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

MBL: Add magnet compensation parameter #4390

Merged
merged 3 commits into from
Sep 22, 2023
Merged

Conversation

sarusani
Copy link
Collaborator

@sarusani sarusani commented Sep 14, 2023

If the user selected 3x3 in the LCD menu he can't set if he wants to use magnet compensation or not. (It's only available for 7x7)
So I added a magnet compensation parameter (M). It is only in use if a 7x7 mesh bed leveling is performed.

@github-actions
Copy link

github-actions bot commented Sep 14, 2023

Target ΔFlash (bytes) ΔSRAM (bytes)
MK3S_MULTILANG 18 0
MK3_MULTILANG 18 0

@sarusani sarusani changed the title MBL: Add magnet compensation parameter & default to EEPROM MBL: Add magnet compensation parameter Sep 14, 2023
allow to enable/disable magnet compensation when 7x7 is in use.
Firmware/Marlin_main.cpp Outdated Show resolved Hide resolved
@3d-gussner 3d-gussner added this to the FW 3.14.0 milestone Sep 15, 2023
Copy link
Collaborator

@leptun leptun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, but I'm a bit on the fence about choosing M as the argument letter. Don't really like that part, but it should work now that only the first letter in the command is interpreted as the "command", so M is properly identified as an argument.

@sarusani
Copy link
Collaborator Author

I feel the same way. I just wasn't able to come up with something else that remotely makes sense and doesn't clash with the other arguments. I was thinking about E for "elimination". But outside of the code, the feature is always referred to as "magnet compensation".

@3d-gussner
Copy link
Collaborator

@sarusani Had to fix the merge conflict as I merged 4391 first.

@3d-gussner 3d-gussner merged commit f8c64de into prusa3d:MK3 Sep 22, 2023
@sarusani sarusani deleted the mbl_mag_comp branch September 22, 2023 16:23
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.

4 participants