Replies: 10 comments 6 replies
-
|
You are trying to save pid values that are already saved in an include elsewhere. It is telling you that in the corresponding [include "filenamehere".cfg] there is a section called [extruder] that has an option listed as "control" likely "control: pid" I will post a solution to this tomorrow. Also, you will want to do your pid tune at your normal print temp with your typical fan speed set. Example: if you mostly print PLA, then pid tune to your print temp and set the fan to 100% prior to running the test. If you plan to use a hotend sock, ensure it is installed prior to starting the test. Take care |
Beta Was this translation helpful? Give feedback.
-
|
This doesn't make sense. Why would a fresh install already have "control" listed? It's commented out of the template, and the instructions say not to include it yet until you do a PID tuning. |
Beta Was this translation helpful? Give feedback.
-
|
[extruder] I added the items above to my p[rinter.cfg and then manually added the values after the last line where it says not to add things. This is not advised and should not be a primary method to resolve the issue If any of the format in the bottom section is wrong, it will ignore the entire section so ensure that you respect spacing and format. TBH, you shouldn't be adding things here manually, but it worked for me. Part of the issue is that things that are called out in the cfgs that aren't printer.cfg are read only and can't be edited. I think this causes some complications. My solution was to copy the offended cfgs and create {custom_insertcfgnamehere}.cfg files and call those in the includes instead of the versions from the git. |
Beta Was this translation helpful? Give feedback.
-
Have you been able to correct your issue with saving PID? |
Beta Was this translation helpful? Give feedback.
-
|
Hello, same issue for me. |
Beta Was this translation helpful? Give feedback.
-
|
Same issue with the bed PID |
Beta Was this translation helpful? Give feedback.
-
|
To solve the SAVE_CONFIG recording for both PID I simply added in the config.cfg file the bellow lines with the PID parameter returned by my printer. [extruder] [heater_bed] This is the correct way to proceed? |
Beta Was this translation helpful? Give feedback.
-
|
This will give previously established values but I still run into saving
issues.
What I did was create all new cfg files to get around some of the saving
and read only issues.
This repo is a good starting point but I would recommend learning to bend
it to your will
…On Thu, Jan 4, 2024, 09:36 Olivier HUBERT ***@***.***> wrote:
To solve the SAVE_CONFIG recording for both PID I simply added in the
config.cfg file the bellow lines with the PID parameter returned by my
printer.
[extruder]
pid_Kp: 23.862
pid_Ki: 1.020
pid_Kd: 139.595
[heater_bed]
pid_Kp=32.683
pid_Ki=2.758
pid_Kd=96.823
This is the correct way to proceed?
—
Reply to this email directly, view it on GitHub
<#24 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABGDEE6UPKNU7XU7PF766LTYM245PAVCNFSM6AAAAAAW4KFQ5GVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM4DAMJTGQZTA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
After you do the calibration and before save, you will have the values printed in the console. You need to save those to printer.cfg on your own, and that will override those in stock config. I've updated the defaults to make it clearer. |
Beta Was this translation helpful? Give feedback.
-
|
Klipper supports multiple definitions of same config (to extend an override), but not when using SAVE_CONMFIG command. Master was updated and from now on everyone should be good. |
Beta Was this translation helpful? Give feedback.
-
|
Charmin,
Tbh, this repo is not very active.
I have a working mk3 running klipper that is originally based on this repo.
If you wanted to work together to do some better documentation and
establish a better baseline, id be interested
…On Wed, Feb 7, 2024, 11:41 charminULTRA ***@***.***> wrote:
This doesn't make sense. Why would a fresh install already have control
listed? It's no where in the config file template, and no where in the
extruder files.
—
Reply to this email directly, view it on GitHub
<#24 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABGDEE6IQ2ULHLL4S63NBRDYSOVB5AVCNFSM6AAAAAAW4KFQ5GVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM4DGOJXGQZDG>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
Cool, I'll try to get some PRs submitted when I get the chance this week. |
Beta Was this translation helpful? Give feedback.
-
|
If you are in the klipper discord, hit me up.
@confuzled
…On Wed, Feb 7, 2024, 12:07 charminULTRA ***@***.***> wrote:
Cool, I'll try to get some PRs submitted when I get the chance this week.
—
Reply to this email directly, view it on GitHub
<#24 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABGDEEY3KKQS4QNPZ2RIUK3YSOYEBAVCNFSM6AAAAAAW4KFQ5GVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM4DGOJXG4YDC>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
??? |
Beta Was this translation helpful? Give feedback.
-
|
Sorry mate. Your name is my discord name and I am on Mobile. Didn't realize
it @ you
…On Wed, Feb 7, 2024, 13:14 Confuzled ***@***.***> wrote:
???
—
Reply to this email directly, view it on GitHub
<#24 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABGDEEZCWYPY2R6MRYDTA6LYSPAA7AVCNFSM6AAAAAAW4KFQ5GVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM4DGOJYGQZDO>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
No worries |
Beta Was this translation helpful? Give feedback.
-
After running PID_CALIBRATE HEATER=extruder Target=170 and trying to run SAVE_CONFIG I get the following error
SAVE_CONFIG section 'extruder' option 'control' conflicts with included value
Beta Was this translation helpful? Give feedback.
All reactions