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

Option to install accelerometer dependencies #534

Open
CODeRUS opened this issue Sep 21, 2024 · 2 comments
Open

Option to install accelerometer dependencies #534

CODeRUS opened this issue Sep 21, 2024 · 2 comments
Labels
hacktoberfest Help wanted Scope: Components Status: Available Something that was accepted and can be worked on Type: Feature Request New feature or request
Milestone

Comments

@CODeRUS
Copy link
Contributor

CODeRUS commented Sep 21, 2024

Is your feature request related to a problem? Please describe

Adding accelerometer requires manual packages installation

Describe the solution you'd like

kiauh could offer an option to install required packages, if needed

see: https://www.klipper3d.org/Measuring_Resonances.html#software-installation

Describe alternatives you've considered

manual installation is still manual

Additional information

No response

@dw-0 dw-0 added Type: Feature Request New feature or request Status: Available Something that was accepted and can be worked on Scope: Components labels Sep 21, 2024
@dw-0
Copy link
Owner

dw-0 commented Sep 21, 2024

Hi, thanks for the request. What would be the best place to add the feature to? Should a prompt during the Klipper installation ask the user if accelerometer dependencies should be installed? Or maybe a new item in the Advanced Menu. Afaik the installation of numpy can fail, especially on weaker hardware with not enough RAM. So maybe it is not a bad idea to decouple those dependency installtion from the regular klipper setup. Another benefit would be, that the implementation should be a bit simpler.

@dw-0 dw-0 added this to the next milestone Sep 21, 2024
@CODeRUS
Copy link
Contributor Author

CODeRUS commented Sep 21, 2024

Something in Advanced menu should be a good fit, someone might not want to use accelerometer initially but later decide to get one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest Help wanted Scope: Components Status: Available Something that was accepted and can be worked on Type: Feature Request New feature or request
Projects
Development

No branches or pull requests

2 participants