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

Implement boresight commands for SAT #73

Closed
kmharrington opened this issue Jul 18, 2023 · 2 comments · Fixed by #77
Closed

Implement boresight commands for SAT #73

kmharrington opened this issue Jul 18, 2023 · 2 comments · Fixed by #77
Labels
enhancement New feature or request

Comments

@kmharrington
Copy link
Member

kmharrington commented Jul 18, 2023

Adding to the list of things: I have noticed that boresight positions aren't in the "move to" command.

EDIT: Co-rotator feature request split to #76.

@BrianJKoopman BrianJKoopman added the enhancement New feature or request label Jul 19, 2023
@BrianJKoopman
Copy link
Member

These will also need to be different for the LAT and SAT.

Thanks, Katie. How do these differ between the platforms? The ACU agent is common and has a single set_boresight() command, which I think might control the 3rd axis in either case.

@kmharrington
Copy link
Member Author

Oh interesting that the commands are the same. Optically they'll be fairly different since the co-rotator isn't exactly a boresight rotation. I would imagine for the LAT we'd want functions that did move_to(az, el, corotate=True) and then an option to move the co-rotator independently. For the SAT the boresight really is a third axis and the use-case is more move_to(az, el, bore)

@BrianJKoopman BrianJKoopman changed the title Boresight and co-rotator commands Implement boresight commands Aug 3, 2023
@BrianJKoopman BrianJKoopman changed the title Implement boresight commands Implement boresight commands for SAT Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants