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

ACU -- updates to work with LAT #526

Merged
merged 8 commits into from
Sep 19, 2023
Merged

ACU -- updates to work with LAT #526

merged 8 commits into from
Sep 19, 2023

Conversation

mhasself
Copy link
Member

Description

Enables basic LAT co-rotator control and readout. A couple of minor improvements come along for the ride.

These are fairly minor changes; there are more important fixes in the latest versions of soaculib and ocs-web. Those should be up to date to take full advantage of LAT.

How Has This Been Tested?

This is the result of direct testing on the LAT.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@BrianJKoopman BrianJKoopman self-requested a review September 13, 2023 21:15
@mhasself
Copy link
Member Author

I have just added the "az_drift" parameter to generate_scan.

Also this whole deal has now been tested on a SATP without any issues.

Copy link
Member

@BrianJKoopman BrianJKoopman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just one quick docs request/question.

socs/agents/acu/agent.py Show resolved Hide resolved
@BrianJKoopman
Copy link
Member

Thanks for the update, will merge once checks pass.

@BrianJKoopman BrianJKoopman merged commit 385d340 into main Sep 19, 2023
2 checks passed
@BrianJKoopman BrianJKoopman deleted the mhasse/acu4lat branch September 19, 2023 18:50
hnakata-JP pushed a commit that referenced this pull request Apr 12, 2024
* ACU: cmdline switch to not start monitor/broadcast

* ACU: corotator data are tagged as "Corotator" instead of "Axis3"

Other fixes to work on the LAT; some simplifications

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* ACU: fix turn-around time computation

The scan generator code was adding an extra "step_time" (usually 1s)
to each turn-around.

* ACU: remove az_min / az_max in preparation for az_drift

* ACU: add az_drift parameter to scan generator

* ACU: add az_drift to agent

* ACU: document startup arg

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

2 participants