Rappetor / Sovol-SV08-Mainline

Getting the Sovol SV08 onto mainline Klipper
GNU General Public License v3.0
95 stars 26 forks source link

'probe_pressure.py' and 'z_offset_calibration.py' #55

Closed Mad-Virus closed 1 month ago

Mad-Virus commented 1 month ago

Hi,

First of all, "THANK YOU" for that nice Guide. 💯
It is idiotproof. I prooved it. 😄

So i decidet to make some more horsepowers and want to remove the Pressure Sensor. Because of physical Endstops and Klicky.

That made me a hard Time. Commenting out the [probe_pressure] throwed me errors and want to force me letting the entry activated while anyone on Sovol Klipper doesn't seem to have the Problem. Because i want to reuse the PIN and like to clean up after me i digged deeper in it. I checkt out your both modified versions of Sovols Breakouts. I am not a programmer or something but text in it reads like [put_it_back_in_or_Fail_you_freakin_noob].

After testing around about 1 Milion Hours with activation and disabling, renaming and stuff i was shure that the modified version will not work without the [probe pressure] entry.

So renamed back z_offset_calibration.py and disabled [probe_pressure] made me see that i was right, because it directly told me to [put_it_back_in_or_Fail_you_freakin_noob]. So i decidet to rename also the 2nd File and uncomment [probe_pressure] and [z_offset_calibration]. Printer restarts like nothing happend the 2 million hours bevore.

It looks like that the PS is optional in the Sovol Scripts, but not in yours, and you have to disable both.

What i dont know, because i am realy a braindead, do i need this stuff in any way if i not use the Failprobe and the PS from them?

Greetings Alex

Rappetor commented 1 month ago

If you remove the pressure probe from the config then the sovol ZoffsetCalibration will not work anymore. If you want to disable this you need to comment out both [probe_pressure] and [z_offset_calibration] in the printer config.

Z offset calibration will call the pressure probe in both the sovol and the fixed script. They go hand in hand.

Mad-Virus commented 1 month ago

Thanks.

I removed both now because it seems they are no longer needed with klicky.

Rappetor commented 1 month ago

Awesome! I say this can be closed. Just had to comment them both out! 😄