Yes, makes sense, lets NOT use the value directly from the configuration define but modify it before setting it. This will not confuse people (including me) at all!

2012-11-09 22:36:31 +01:00 · 2012-11-09 22:36:31 +01:00 · 625ac81acc
parent ffc69537aa
commit 625ac81acc
1 changed files with 2 additions and 2 deletions
--- a/Marlin/ConfigurationStore.cpp
+++ b/Marlin/ConfigurationStore.cpp
@ -227,8 +227,8 @@ void Config_ResetDefault()
 #endif
 #ifdef PIDTEMP
    Kp = DEFAULT_Kp;
-    Ki = DEFAULT_Ki;
-    Kd = DEFAULT_Kp;
+    Ki = (DEFAULT_Ki*PID_dT);
+    Kd = (DEFAULT_Kp/PID_dT);
 #ifdef PID_ADD_EXTRUSION_RATE
    Kc = DEFAULT_Kc;
 #endif//PID_ADD_EXTRUSION_RATE