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

interface: start pulling out the stage concept #34

Merged
merged 2 commits into from
Dec 30, 2024
Merged

Conversation

ianohara
Copy link
Collaborator

@ianohara ianohara commented Dec 6, 2024

This is starting work on the "X, Y, Z Motion stage" task in https://www.notion.so/cephla/Headless-Mode-and-Scripting-Design-1452dfbf6ae480c3b444ee0847f532a5 .

It's gonna be a tricky one - we have stage-like code in a lot of places. But the net result is that we will end up deleting a ton of tricky code (eg: all the unit conversion stuff) from a bunch of places, and instead have it all focused in one location.

The hard part isn't the stage implementation, it's finding all the places we do stage-like stuff, fixing them, and making sure nothing is broken in the process.

@ianohara ianohara force-pushed the ian-microcrontroller-simulation branch from 2911bad to d2a945f Compare December 9, 2024 21:33
@ianohara ianohara changed the base branch from ian-microcrontroller-simulation to master December 9, 2024 21:39
@ianohara ianohara force-pushed the ian-stage-breakout branch 2 times, most recently from 4005e01 to 73fd655 Compare December 10, 2024 00:07
@ianohara ianohara marked this pull request as ready for review December 10, 2024 01:23
@ianohara ianohara merged commit d9fb440 into master Dec 30, 2024
1 of 2 checks passed
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.

1 participant