Skip to content

Better error reporting #2585

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Better error reporting #2585

wants to merge 6 commits into from

Conversation

@t-b
Copy link
Collaborator

@t-b t-b commented Nov 28, 2025

Related to #2568

t-b added 6 commits November 28, 2025 20:22
@t-b
ASSERT/ASSERT_TS: Add AssertOnAndClearRTError in fail case
0e34b52 
This should make finding the place of lingering RTEs easier.
@t-b
Catch lingering RTEs for labnotebook reading/writing
c02b5fa 
This would have helped in debugging issue #2568.
@t-b
ASSERT/ASSERT_TS: Output time with milliseconds precision
f74ca5d
@t-b
ASSERT: Output fifo position
7b7dab2
@t-b
GetLastSetting: Pass settingCol into GetLastSettingNoCache
0e7a4e6 
This then takes the fast path as we don't need to determine it again.
@t-b
AFM_CallAnalysisFunctions: Distinguish between RTEs and Aborts in cat…
a30e6fd 
…ch case

And also tone down the message do the user.
@t-b t-b self-assigned this Nov 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@timjarsky timjarsky Awaiting requested review from timjarsky timjarsky will be requested when the pull request is marked ready for review timjarsky is a code owner

@MichaelHuth MichaelHuth Awaiting requested review from MichaelHuth MichaelHuth will be requested when the pull request is marked ready for review MichaelHuth is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@t-b t-b

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@t-b