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
[1.1.x] G33 magic numbers #8173
[1.1.x] G33 magic numbers #8173
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good. Just a couple tweaks requested. Same will apply to the 2.0.x version.
Marlin/enum.h
Outdated
__C = _BC + _7P_STEP, | ||
_CA = __C + _7P_STEP, | ||
NPP = _7P_STEP * 6 // number of calibration points on the radius | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not really the proper use of enum
. I would recommend doing these as constexpr uint8_t
instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As these are only used by G33
, they would be better defined just above the G33
function in the .cpp file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reworked this
doc is changed in #89 to reflect extra probe grids |
* [1.1.x] G33 magic numbers * oops * comments * oops * warning * better comment section * remarks * extra grids
* [1.1.x] G33 magic numbers * oops * comments * oops * warning * better comment section * remarks * extra grids
see remarks in #8137