Temperature reporting conflict.
Opened this issue · 5 comments
Bug report
Temperature reporting conflict.
Description
After a corner or two adjustments become "huge" By process of elimination, I believe the temperature reports in the com stream are confusing AutoBim.
Steps to reproduce
Happens by default in my installation/environment. It is necessary to issue an M155 S(large num of seconds) then remember to set back to M155 S3 once complete.
Expected outcome
Had this issue with another bed leveler plugin. (Bed Visualizer). In that case the plugin provides a place to auto send M155 before and after the process.
Actual outcome
Will never complete because of the confusion.
Describe what actually happened.
Environment (leave empty if unknown or not relevant)
- OctoPrint version: 1.6.1
- Octopi: 0.18
- Python version: 3.7.3
- Marlin version: 2.0.9
- Probe (e.g. BL-Touch
- Printer Ender 3 Pro with 4.2.7 board
- Browser (especially useful on UI issues): Chrome/93.0.4577.63 Safari/537.36
For UI issues a screenshot always helps.
Feature
A place to indicate the number of seconds to disable temperature reporting, then reenable automatically on exit or user cancel
Please note by far the quickest way to get a new feature is to file a Pull Request.
Just describe what you would like AutoBim as detailed as you can.
Question
Make sure your question is not covered by the Troubleshooting section.
I did not see this issue in troubleshooting or open issues. Sorry if it is addresssed.
Else: just fire away. But keep in mind: the more detail, the easier it is to answer 😃.
@garciaramondallas S**t. Are you running one of those Creality firmwares that need https://plugins.octoprint.org/plugins/ender3v2tempfix/ to work properly?
Yes sir that is correct. I had to install the temp fix plugin some time ago to allow something else to operate.
However since then I have updated to Creality's latest board 4.2.7. and I have learned how to compile my own FW load. Let me look through the Marlin 2.0.9.1 branch source code. If it has a published or included fix I can try to patch it there. This will allow me to disable the temp fix plugin.
Ramon
All good.
The Marlin 2.0.9.1 branch does seem to include the fix for double temp reporting issue.
The octoprint temp plugin fix is no longer needed and disabled. The terminal temperature reports are 'normal' looking.
AutoBim can complete the leveling as intended
@garciaramondallas happy to hear it worked 😃
I'll reopen this issue anyways, as I think I'll need to deal with Creality's crappy firmwares sooner or later anyway. If you don't want to get any updates on this feel free to use the "Unsubscribe" button above.
@garciaramondallas one more thing: would it be possible for you to provide a full log from OctoPrint's "Terminal" tab of such a failed run on the crappy Creality firmware? I do not own a Creality v4 board (went for an SKR Mini E3 v2), so it is really hard for me to debug this issue. I suspect this is due to Creality's crap, not Marlin itself.
If it is too much of a hassle: nevermind, I guess someone else will complain eventually 😄