Skip to content

Improvements to experiment error handling#482

Open
Himorask wants to merge 7 commits intodevelopfrom
improve-error-handling
Open

Improvements to experiment error handling#482
Himorask wants to merge 7 commits intodevelopfrom
improve-error-handling

Conversation

@Himorask
Copy link
Copy Markdown
Contributor

This is focused on experiment.py and qubit_exp.py, other specialized experiments may need to have similar changes made to match the behavior.
Some key changes:
-If one instrument fails to connect, disconnect from all previously connected instruments
-Report which instrument failed to connect/disconnect
-Check that listeners/dig_exit exist before trying to modify them in shutdown code

Minor changes to how instrs_connected is handled
dig_exit attribute check
Minor changes to a failure block
Fixed the mistake I was making with pass
Copy link
Copy Markdown
Member

@grahamrow grahamrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me!

@grahamrow
Copy link
Copy Markdown
Member

grahamrow commented Apr 14, 2021

@Himorask a lot of this was already added in #486, but we'd like to keep some of the stuff in the shutdown methods in particular. Can you help resolve the conflicts here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants