PID_CALIBRATE advice test max power after calibration#7209
Open
nefelim4ag wants to merge 2 commits intoKlipper3d:masterfrom
Open
PID_CALIBRATE advice test max power after calibration#7209nefelim4ag wants to merge 2 commits intoKlipper3d:masterfrom
nefelim4ag wants to merge 2 commits intoKlipper3d:masterfrom
Conversation
Signed-off-by: Timofey Titovets <nefelim4ag@gmail.com>
For high power heaters current calibration might give too high ID of PID. Detect that by time ratios and suggest to rerun the calibration at different max power. And add a notice about the too-high heating time. Signed-off-by: Timofey Titovets <nefelim4ag@gmail.com>
79077c5 to
eb079f2
Compare
|
PID changes without restarting WHEN |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Right now, some heaters can produce suboptimal pid values after calibration, they are suboptimal because they produce large oscillations during standby or printing. Often, it looks like
IandDare large, and decreasing them helps.That happens, that the calibration of the heater when the time of heating is close (equal) to the time of cooling produces less aggressive PID values.
Those values in the end should be less annoying and less often produce several degrees of oscillations, which can block the machine on
M109indefinitely.This PR adds a new param to the
PID_CALIBRATE,MAX_POWER, which allows adjusting power during the calibration.On top, here is a small analysis section, which advises/guess
MAX_POWERvalue to use next.The point is, reduction might be necessary during calibration, but it is not necessary to reduce power for real printing.
*Based on Kevin's idea, I've only implemented it and gathered some testing data before
Thanks,
-Timofey
Testing:
Graph with default PIDs at steady state:

With recommended power:
So, for me, there is no significant difference.
I did try to calibrate my bed with decreased power (18%), but it seems to make
Itoo low (0.064), so it has too slow heating and overshoots more than I prefer it to.