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.

Sign up for GitHub

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

Jump to bottom

Remove initial_xyz #541

Open
DominicOram opened this issue Sep 30, 2024 · 1 comment
Open

Remove initial_xyz #541

DominicOram opened this issue Sep 30, 2024 · 1 comment
Labels
hyperion

Comments

@DominicOram
Copy link
Contributor

Following #538 initial_xyz is no longer really used other than by the panda setup, which only really uses the x. Instead the panda setup should just read the x and otherwise we shouldn't be reading the initial at all.

Acceptance Criteria

  • The code reading initial_xyz in the xrc is removed
@DominicOram DominicOram mentioned this issue Sep 30, 2024
Open
@DominicOram
Copy link
Contributor Author

May disappear with the multipin work anyway

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hyperion
Projects
Hyperion
Status: Todo This Sprint
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DominicOram