-
-
Notifications
You must be signed in to change notification settings - Fork 19.2k
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
[FR] Auto Report Redundant Sensor option #23997
Comments
Have you tried your patch? If it works, it'd be faster if you submitted a Pull Request so it can be reviewed. |
This was simple enough to implement on the fly. See the PR linked above. |
@thinkyhead Thank you! Your solution is a little more elegant than mine. I appreciate the help! |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Is your feature request related to a problem? Please describe.
Marlin has the framework for this available but reporting the redundant sensor is hardcoded to false
Are you looking for hardware support?
N/A
Describe the feature you want
In temperature.h
Add a check for the auto-report redundant flag. If it is enabled, print_heater_states would have the include_r flag set to true if it is not, it would have it set to false.
In Configuration_adv.h
Add a condition that if AUTO_REPORT_TEMPERATURES is enabled and TEMP_SENSOR_REDUNDANT is defined. If those conditions are met, then AUTO_REPORT_REDUNDANT can be enabled. By default, AUTO_REPORT_REDUNDANT would be commented out and would be an option to be enabled when configuring Marlin
Additional context
No response
The text was updated successfully, but these errors were encountered: