Skip to content
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

Assertion message lost #158

Closed
csatt opened this issue Dec 4, 2020 · 0 comments
Closed

Assertion message lost #158

csatt opened this issue Dec 4, 2020 · 0 comments
Assignees
@csatt
Labels
bug

Comments

@csatt
Copy link
Owner

csatt commented Dec 4, 2020

If the add_reference_curve() method fails due to an assert, the reason for the assertion is lost.

In particular, if the PV name is not found in the CSV file, there is no indication to the user that this is the problem. The user just sees a dialog saying that the PV modeling failed. In fact, the two possible reasons that the dialog lists do not include this as a possibility.
@csatt csatt self-assigned this Dec 4, 2020
@csatt csatt added the bug label Dec 4, 2020
csatt added a commit that referenced this issue Dec 4, 2020
@csatt
Capture assert message in plot_results (Issue #158)
ba5889f 
The assertion message is captured in an IV_Swinger2 property so the GUI code can display it in the error dialog.
@csatt csatt closed this as completed Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@csatt csatt

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@csatt