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

[PREVIEW] Initial focuser support added to indi_celestron_aux #862

Merged
merged 6 commits into from
Jan 9, 2024

Conversation

ijessen
Copy link
Contributor

@ijessen ijessen commented Dec 3, 2023

This is functional - but testing has been limited. Most importantly, I haven't tested with an uncalibrated focuser.

I took a slightly different approach than the focuser implementation in the core celestrongps driver - namely I adopted the interface guidance that the absolute position property value ranges from 0 to max. I do this by applying an offset (the min value from calibration), and therefore the abs position value from the driver will be offset from the abs position displayed on the HC. I also skipped relative positioning because I don't see a use case for it when absolute is available.

Backlash not yet implemented.

Astyle not (yet) applied.

@knro
Copy link
Collaborator

knro commented Dec 11, 2023

Thanks! So what's the procedure exactly with a new uncalibrated focuser? Should there be a similar calibration workflow like in the HC driver?

@ijessen
Copy link
Contributor Author

ijessen commented Dec 11, 2023

As is there's nothing here to initiate a calibration - the interface isn't loaded unless a valid calibration is already present (you would have to use an HC or other driver to launch the calibration routine). That said it's easy enough to add (and I see that there's an example to copy from in the dedicated celestron focuser driver in the core repo). I'll have some time later in the week to fiddle with it.

@knro
Copy link
Collaborator

knro commented Dec 17, 2023

@ijessen Did you test this? any issues?

@ijessen
Copy link
Contributor Author

ijessen commented Dec 19, 2023

@ijessen Did you test this? any issues?

Haven't gotten around to adding in calibration support yet. I was waiting until I get to that before taking the PR out of draft.
That said I've been using this in the field for a couple weeks and it's been rock solid.

@knro
Copy link
Collaborator

knro commented Dec 20, 2023

Awesome, looking forward to the final driver!

@knro
Copy link
Collaborator

knro commented Jan 5, 2024

@ijessen Any update? should we merge this as-is to get some community feedback? any show stoppers?

@knro knro marked this pull request as ready for review January 9, 2024 09:14
@knro knro merged commit 760bc4e into indilib:master Jan 9, 2024
4 checks passed
@ijessen ijessen deleted the celestron-aux-focuser branch January 24, 2024 17:46
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