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

Scanning for focus when original offset is close to zero #97

Closed
seichhorn opened this issue Jan 22, 2020 · 5 comments
Closed

Scanning for focus when original offset is close to zero #97

seichhorn opened this issue Jan 22, 2020 · 5 comments

Comments

@seichhorn
Copy link
Contributor

I'm finding that when I lock my focus lock with an offset very close to zero, if I lose focus and enter the scan_focus routine I end up erroneously accepting the first measurement. This seems to be because 1) the uc480camera is returning total_good = 0, and 2) the difference between my IR spots returns as 0 owing to no good fits. The difference of 0 passes the threshold for the difference relative to the offset in the handleQPDUpdate step of the ScanMixIn class. Is there a better value to return here instead of 0?

@jeffmoffitt
Copy link
Collaborator

Do you have this problem with the current version? I encountered what I think may be the same problem, and we pushed a fix recently.

@seichhorn
Copy link
Contributor Author

Thanks, I was on another branch and missed that commit, but it seems to have fixed my issue.

@HazenBabcock
Copy link
Member

HazenBabcock commented Jan 22, 2020

I'm still puzzled why the "sum" check is not catching this. When there are no spots on the camera the "sum" signal should be below "minimum_sum". Does no one set "minimum_sum" properly? Or does this just not work for some other reason?

@seichhorn
Copy link
Contributor Author

My focus lock sum that is only ~2-fold lower when my spots are badly out of focus/undetected, and couldn't find a way to set it that guaranteed I didn't erroneously pass the minimum_sum threshold during the scan despite being out of focus.

@HazenBabcock
Copy link
Member

Okay, thanks.

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

No branches or pull requests

3 participants