asaril/marlin2_i2
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Apply minimum C in M303 (#13113)

Browse source 
Apply minimum `C` in `M303`. `PID_autotune()` needs to be called with a 'C'ycles-parameter >2 to give meaningful (different from 0) PID-factors. Therefore silently raise C to 3. Fixing one aspect of #13104
This commit is contained in:
AnHardt 2019-02-10 10:54:45 +01:00 committed by Scott Lahteine
parent 5a316664e5
commit 0e37fe8776
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Marlin/src/gcode/temperature/M303.cpp
View file

@ -32,7 +32,7 @@
*
* S<temperature> sets the target temperature. (default 150C / 70C)
* E<extruder> (-1 for the bed) (default 0)
* C<cycles>
* C<cycles> Minimum 3. Default 5.
* U<bool> with a non-zero value will apply the result to current settings
*/
void GcodeSuite::M303() {

4
Marlin/src/module/temperature.cpp
View file

@ -330,6 +330,8 @@ uint8_t Temperature::soft_pwm_amount[HOTENDS];
*
* Alternately heat and cool the nozzle, observing its behavior to
* determine the best PID values to achieve a stable temperature.
* Needs sufficient heater power to make some overshoot at target
* temperature to succeed.
*/
void Temperature::PID_autotune(const float &target, const int8_t heater, const int8_t ncycles, const bool set_result/*=false*/) {
float current = 0.0;
@ -540,7 +542,7 @@ uint8_t Temperature::soft_pwm_amount[HOTENDS];
break;
}
if (cycles > ncycles) {
if (cycles > ncycles && cycles > 2) {
SERIAL_ECHOLNPGM(MSG_PID_AUTOTUNE_FINISHED);
#if HAS_PID_FOR_BOTH