-
Notifications
You must be signed in to change notification settings - Fork 3
Draft of on-sky setup documentation #16
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
base: main
Are you sure you want to change the base?
Draft of on-sky setup documentation #16
Conversation
joshua-liberman
commented
Apr 19, 2024
- Docs describing how to set up the AO system on-sky, prior to starting science observations
|
Thanks for this, @jlibermann. It looks like something funky happened with index.rst. It could be windows line endings. Could you correct that so the only edited lines are the one you actually added? |
jaredmales
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job @jlibermann. A few details to clean up. I think we want to reorganize a little to add an alignment procedure, based on the one I put on slack and is pinned, as a separate thing that happens before acquisition and loop-closing. You can get higher quality figures for the guis using your remote operations ability and screen-shotting them.
operating/on_sky.rst
Outdated
| 1. Modulating the WFS | ||
|
|
||
| * Set the FPS on the ``camwfsctrl`` GUI to ``2000`` | ||
| * In CursesINDI, type ``fxngensync`` and toggle the state to ``On`` (see the `cursesINDI guide <./software/guis/cursesINDI>`__ if you need a refresher) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
state of what? it's C1outp
operating/on_sky.rst
Outdated
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
|
|
||
| #. In the web interface, set ``fwpupil`` to ``bump-mask``. | ||
| #. Now use the ``Pupil Plane`` buttons in the ``coronAlignGUI`` to align the bump mask on the pupil. Set the step size to ``100`` and press the down arrow 3 times. This is typically enough to align the bump mask. However, if the bump mask still appears slightly misaligned, reduce the step size to ``5`` and use the left/right arrow keys for fine adjustments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is correct, but assumes that the alignment was already done on the soruce to get the mask close. just re-making the point that we need a separate alignment procedure (like the one that is pinned on slack)
operating/on_sky.rst
Outdated
| Closing the Pupil Alignment Loop | ||
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
|
|
||
| #. In the ``Pupil Alignment`` GUI, use the keypad in the lower-righthand corner to drive the x and y avg values to 0. After each translation, wait a few seconds for the readings to stabilize. Also note that the x-axis is much more sensitive than the y-axis. When moving x-axis avg to 0, use step sizes of about 0.1. When moving y-axis to 0, higher step sizes can be used. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the idea is correct. It's really something like "when x is within 0.2, steps sizes of 0.05 should be used. The y-axis steps size can remain 0.1"
operating/on_sky.rst
Outdated
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
|
|
||
| #. In the ``Pupil Alignment`` GUI, use the keypad in the lower-righthand corner to drive the x and y avg values to 0. After each translation, wait a few seconds for the readings to stabilize. Also note that the x-axis is much more sensitive than the y-axis. When moving x-axis avg to 0, use step sizes of about 0.1. When moving y-axis to 0, higher step sizes can be used. | ||
| #. Once the x, y avgs are at roughly 0, set the pupil alignment loop gain to 0.1 and turn on the pupil alignment loop. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change "roughly 0" to "0 +/- 0.1"
…rman/handbook into onsky-operation-docs
|
What's going on here? It looks like a lot more changes and some regressions (introducing old typos etc)? |
57991d3 to
ed5dc34
Compare