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

Make align system more dynamic and user-friendly #277

Merged
merged 5 commits into from
Feb 14, 2025
Merged

Make align system more dynamic and user-friendly #277

merged 5 commits into from
Feb 14, 2025

Conversation

brickbots
Copy link
Owner

Based on great feedback from @mrosseel this is a PR to polish up the Align feature:

  • Use IMU updated camera center position for chart while not in star selection mode
  • Allow zoom in/out when in star selection mode
  • Use standard reticle to indicate current alignment point before entering star select mode
  • Provide prompts along the bottom of the screen to indicate next steps

I tried to use the IMU during the entire process, but it made it much more confusing as the selection point would move to the star's location when the arrow was pressed, but this would almost alwasys be a little different in the next moment due to the imu motion from interacting with the PiFinder. This would make it seem like it was not selecting the star, but was slightly offset. Tying the location to the camera solve during star selection makes for a nicer experience as the selected star is well indicated and the set of stars is stable for selection.

To help guide the process and avoid user confusion, the chart starts dynamic using the IMU and shows the current alignment using the normal reticle pattern. A prompt is displayed along the bottom to guide the user through and should, hopefully, make the process more understandable.

@brickbots brickbots requested a review from mrosseel February 9, 2025 19:22
@brickbots brickbots merged commit 133be6b into main Feb 14, 2025
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