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.

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

Creating multiple top level runners with for loop and using a single applet leads to value error #236

Open
dougalmain1 opened this issue Feb 25, 2021 · 5 comments
Labels
A-applet Involves the ndscan plot applets

Comments

@dougalmain1
Copy link

The code runs a 2D Ramsey scan for multiple transitions. For each transition, a new instance of TopLevelRunner is called. The applet does not seem to be being reset and we get the following message.

ndscan_issue

The experiment does not stop upon receiving this message and it appears that apart from the applet not displaying correctly, the code runs fine.

@dougalmain1
Copy link
Author

The issue with the applet display only appears to happen when the scan axes have different ranges.

@dnadlinger
Copy link
Member

@dougalmain24: Do you have a test case to reproduce this? It's fine if it's an internal one, but this would be much easier to debug with a concrete starting point.

@dougalmain24
Copy link

Ah yes, I believe this was the coherence time measurements (hoa2_common.measurements.coherence_times) which we run in nightly, where we have a set of 2d phase/duration scans. The first scan runs normally, the applet looks fine (see below).
image

However, when we run the next scan, which then uses a different duration axis, the applet does not display the data correctly in that the applet seems to use the same duration axis as the first scan (see below)? And we often get points overwriting others (in the applet only, the data and thus the subsequent fitting etc.. itself is fine).
image

@dougalmain24
Copy link

This was a while ago, but I believe this was what the issue was created for. It may well have been user error where I did not use the top level runner correctly.

@dnadlinger dnadlinger added the A-applet Involves the ndscan plot applets label Apr 10, 2024
@dnadlinger
Copy link
Member

This is probably related to #387.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-applet Involves the ndscan plot applets
Projects
None yet
Development

No branches or pull requests

3 participants