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

fix: Use probe-manager for zero ref #149

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

cloakedcode
Copy link
Contributor

With recent changes to Klipper's probe.py, the location of some attributes has changed, in this case, the accessing of [probe] zero_reference_position.

Before this change, setting only the [probe] zero_reference_position and not passing BED_POSITION to CALIBRATE_Z or defining the config option, an error would be thrown saying "Cannot find the bed position!".

With recent changes to Klipper's probe.py, the location of some
attributes has changed, in this case, the accessing of `[probe]`
`zero_reference_position`.

Before this change, setting only the `[probe]` `zero_reference_position`
and not passing `BED_POSITION` to `CALIBRATE_Z` or defining the config
option, an error would be thrown saying "Cannot find the bed position!".
@mojeto
Copy link

mojeto commented Jun 29, 2024

I have tested this code change on my printer and it works as expected.
If merged it fixes #148

@TitusLabs
Copy link
Member

Sorry, I have not seen this. Nice work 👍

@TitusLabs TitusLabs merged commit f0ffc31 into protoloft:development Aug 1, 2024
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.

3 participants