MRX8024 / chopper-resonance-tuner

Registers calibration script for TMC drivers
GNU General Public License v3.0
81 stars 13 forks source link

Error after fresh install #22

Closed ReaperXD1 closed 4 weeks ago

ReaperXD1 commented 1 month ago

Hi there, I have just installed the chopper resonance tuner, however I continue to get this error when attempting to run the macro. image

I don't know what is going on, I've even done the manual installation.

I am running a CB1 if that changes anything.

MRX8024 commented 1 month ago

Hi, pin a klippy.log

ReaperXD1 commented 1 month ago

Thanks for your quick reply. I've had to compress it as it was over githubs file size limit. klippy (3).zip

MRX8024 commented 1 month ago

Please update to the latest version, I have fixed some possible errors there, and attach the log one more time

ReaperXD1 commented 1 month ago

Still having this issue. Does it matter that I am using the BTT Klipper fork required for the new Eddy sensor? image klippy.log

MRX8024 commented 1 month ago

Try to install original klipper, which already support eddy. But it’s strange if btt broke something in their fork. And can you attach link to this fork please

ReaperXD1 commented 1 month ago

Original Klipper doesn't support the BTT Eddy unfortunately.

https://github.com/bigtreetech/Eddy/tree/master - Eddy github https://github.com/bigtreetech/klipper - BTT Klipper

MRX8024 commented 1 month ago

https://www.klipper3d.org/Config_Reference.html#probe_eddy_current

ReaperXD1 commented 1 month ago

I will try to get it working, give me a little :)

ReaperXD1 commented 1 month ago

Hi there, same story, on latest Klipper klippy (2).log

MRX8024 commented 1 month ago

Since jinja2 cannot show the exact location of the error, I placed tags in the code. Please replace the macro, for example by replacing 'include' line, and attach the log or photo of terminal chopper_tune_test.cfg.txt

ReaperXD1 commented 1 month ago

Hi, attached is the new Klippy log. klippy (4).log

MRX8024 commented 1 month ago

I think I found a mistake.. Try this - chopper_tune_test.cfg.txt

ReaperXD1 commented 1 month ago

Hi there I'll give it a shot now!

ReaperXD1 commented 1 month ago

It works! Thank you so much for your work! What was the issue?

MRX8024 commented 1 month ago

Script was supposed to get the sense_resistor value from config, which was not specified there. Now after c67b6c4 I take it from another place. Thanks for the feedback, if you have no questions left, I think we can close this issue.

-Maxim