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

[rfc] - WAND lock control loop gain #39

Open
Jemetzner opened this issue Jan 22, 2021 · 1 comment
Open

[rfc] - WAND lock control loop gain #39

Jemetzner opened this issue Jan 22, 2021 · 1 comment

Comments

@Jemetzner
Copy link

After reviewing the lock method code have discovered that the control loop has the gain set by the product of the set gain and the poll time. This would seem to suggest this is a integrative term, but would only seem to make sense in the limit of a very slowly converging lock.

I have calibrated our laser controllers to get a proper gain factor and turned this control loop into a simple proportional term by removing this poll time factor. We were hoping that there might be some clarification for why this was the method that was used and if there is a interest to improve the lock to make a proper integrator term.

@hartytp
Copy link
Contributor

hartytp commented Jan 22, 2021

This would seem to suggest this is a integrative term, but would only seem to make sense in the limit of a very slowly converging lock.
I have calibrated our laser controllers to get a proper gain factor and turned this control loop into a simple proportional term by removing this poll time factor. We were hoping that there might be some clarification for why this was the method that was used and if there is a interest to improve the lock to make a proper integrator term.

Yes, currently there is only an integrator, designed for relatively slow feedback loops.

V_error = f_error * gain

I don't think I understand your question. Can you explain what you think a "proper integrator term" and "simple proportional term" would look like in this context?

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

2 participants