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

DACCal.IQ: Handle Pyvisa errors #261

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jwenner
Copy link
Contributor

@jwenner jwenner commented Nov 12, 2015

When the spectrum analayzer resets itself (lots of clickies), it was timing out, causing the IQ calibration to fail. Place call to spectrum analyzer in try/except loop so error can be handled gracefully.

When the spectrum analayzer resets itself (lots of clickies), it was timing out, causing the IQ calibration to fail. Place call to spectrum analyzer in try/except loop so error can be handled gracefully.
@pomalley
Copy link
Contributor

I know it's been a while on this but if you're interested, Jim, we can get this in. Have you tested this? (I'm guessing yes.) I might suggest going for a limited number of tries (5 or 10?) rather than going forever--if something is wrong with the VISA connection rather than just taking too long we don't want to be stuck in a loop.

@ejeffrey
Copy link
Contributor

Yes. I agree, there should be a hard timeout in the loop.

FWIW, there are some alternatives to this, but I think they may be more trouble than they are worth. You can disable auto align while a scan is running, but then you have to poll the device and run the alignment yourself. Alternately, you can query if the device is performing an auto-alignment and avoid the timeout that way, although you have to be careful about race conditions.

According to the manual, one of the triggering causes for the auto align cycle is temperature drift of the instrument, so keeping the door closed on the lab may help.

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.

None yet

3 participants