Closed ispielma closed 2 years ago
I think this bug also affects experiments without waits too, see #1. But it looks like your fix isn't specific to waits which is fantastic. It will be good to see our longest standing bug go away!
@ispielma I'm looking to clear out some old PRs that are ready to go and this one fits the bill. Given what Phil and Chris have said, I'm happy to merge, but figured I confirm with you first that you've been running this edit in your labs and everything is still fine?
David,
Yes, we have been running with this change fo a while.
β Ian
Ian B. Spielman
Fellow, Joint Quantum Institute National Institute of Standards and Technology and the University of Maryland
----- WEB ----- http://ultracold.jqi.umd.edu
----- EMAIL ----- @.***
----- ZOOM ----- https://umd.zoom.us/j/7984811536
----- PHONE ----- (301) 246-2482
----- MAIL ----- UMD: 2207 Computer & Space Sciences Bldg. College Park, MD 20742
NIST: 100 Bureau Drive, Stop 8424 Gaithersburg, MD 20899-8424 USA
----- OFFICE ----- UMD: Physical Sciences Complex, Room 2153 NIST: Building 216, Room B131
On Nov 8, 2021, at 09:50, David Meyer @.***> wrote:
@ispielma I'm looking to clear out some old PRs that are ready to go and this one fits the bill. Given what Phil and Chris have said, I'm happy to merge, but figured I confirm with you first that you've been running this edit in your labs and everything is still fine?
β You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.
Superb! Merging.
This pull request is in response to errors like this that we occasionally see:
It seems that this results from the times produced by summing the waits being wrong and as a result the computed length of the AI is larger than the length of the measurements array. This pull request changes the logic so that the times array is always the same length as the values array and the data array.
I don't understand the underlying hardware issue, but it seems that the times reported by the wait monitor are slightly off in some cases .