prusa3d / Prusa-Firmware-MMU

Other
32 stars 15 forks source link

Idler parked current #218

Closed leptun closed 1 year ago

leptun commented 1 year ago

Please read the commits as I had to make a change to a TMC parameter for this.

MMU-182 MMU-186

leptun commented 1 year ago

After leaving the idler enabled with the parked current, it is only a few degrees warmer than ambient. It's basically cold. Of course, measured using the very reliable and scientific hand method for measuring the temperature

leptun commented 1 year ago

There is however a very quiet high pitched noise from the stepper. I wonder if I went a bit too far with the pwm frequency. I can't really make out if the noise is from the delta PSU or from the idler motor. I'll try tomorrow PWM_FREQ=1 and see how it goes

DRracer commented 1 year ago

@leptun the fact, that you removed Disable in FinishMove and the unit tests still went ok points to some issue with insufficient testing. I'll look into this.

Otherwise the changes are well made and relevant, I have no objections to merging this PR once we prove that it solves the "drifting" issue.

leptun commented 1 year ago

In other news, it was the delta PSU making noise. I was also listening to the wrong steppers... so it is fine with PWM_FREQ=0