VoronDesign / Voron-0

Voron 0 CoreXY 3D Printer design
GNU General Public License v3.0
1.37k stars 386 forks source link

Config - Sensorless Homing Current & Macro Confusion #304

Closed SrgntBallistic closed 1 year ago

SrgntBallistic commented 1 year ago

I had some issues setting up Sensorless Homing for my Formbot V0.2(r0) Kit.

I used the bigtreetech-skr-pico-v1.0.cfg

While also walking through the Voron Docs Initial Startup guide and Klipper TMC Docs.

I first walked through all the instructions in the .cfg file.

I set the motor currents based on the recommended equation in the file.

"# you need to calculate the run_current value using the equation (rated_motor_current * 0.707 = Maximum_run_current) start with a value that is about 60%-70% of your maximum value."

My Moons motor currents are:

Using the above equation and factoring a 65% ( x0.65) starting value, I ended up with the following currents

When I got the the _HOME_X/Y macros I first found the wording a bit strange for changing the XX value. It states:

"{% set HOME_CURRENT = XX %} # replace the XX with a current value that is about 70% less than your run_current value for X and Y "

By 70% less does this mean to set XX to 30% of our calculated run_current? That's what I took it as. .69 * 3 = .21 which is a very low current setting.

After doing so I had a lot of trouble getting a stable Stall Sensitivity as outlined in the Starting Guide.

Taking a look at the macro closer I noticed it's actually taking that XX value and setting it as the HOME_CURRENT value. But then sets the current to HOME_CURRENT * RUN_CURRENT for homing

So it seems to me that it is actually looking for a decimal percentage?

Looking at the macros presented in the Voron Sensorless Guide and the Klipper TMC Sensorless Homing Macros section I noticed that neither do any multiplication. They just directly set the current to a hardcoded value for homing and change it back.

So it's possible I've just really confused myself here and read things wrong. Or is the intention to change the XX value to a percentage that will be multiplied by the RUN_CURRENT to drop the current during homing? Or lastly is the goal to have users manually set a current value that's lower than the set RUN_CURRENT?

kyleisah commented 1 year ago

Hey there, just letting you know I will be taking a look into this and see if there's something better I can come up with that works for newbies and pros.

nemgrea commented 1 year ago

https://github.com/VoronDesign/Voron-0/pull/305

SrgntBallistic commented 1 year ago

Hey there, just letting you know I will be taking a look into this and see if there's something better I can come up with that works for newbies and pros.

Thanks. I definitely saw a couple other folks in the discord channel having questions about that.

One of those things were ppl are likely figuring out what it means and how it works. But the clarity can just short circuit some of the process to get to a working value.

Thanks for the work! The config worked great to get started!