johncarlson21 / SV04-Marlin-2.1.x

Marlin is an optimized firmware for RepRap 3D printers based on the Arduino platform. | Many commercial 3D printers come with Marlin installed. Check with your vendor if you need source code for your specific machine.
http://marlinfw.org
GNU General Public License v3.0
21 stars 7 forks source link

[BUG] Parking head with e.g. M125 leads to head crash #6

Closed endrift closed 1 year ago

endrift commented 1 year ago

When using a G-Code like M125 to pause printing, it will try to park the head all the way to the right...which leads to head 1 crashing into head 2, resulting in a layer shift. I believe this is due to NOZZLE_PARK_POINT using Y_MAX_POS, but I'm not sure--I haven't tested it. In the meantime I'm gonna be setting the park point manually, I suppose.

Version of Marlin Firmware

V2.1.3

cryoglobe commented 1 year ago

I think I got a similar case , but after a power failure while printing in copy mode ...

After restarting the machine , heating , Head 2 was going in its position ( as in the beginning of copy mode ) , but suddenly both head moved to the extreme right , so E2 crashed when trying to go too far , and E1 crashed on E2 Then both heads moved back to the print place , but none in the correct position

Conclusion : print is failed , need to switch off the printer.

johncarlson21 commented 1 year ago

Yes there is an issue with the nozzle park point.

I have this as a fix to be pushed up soon.

There is also a different park point for pause vs advanced pause filament change which I'm looking into.

johncarlson21 commented 1 year ago

I've updated the nozzle park point in version 2.1.3.1

endrift commented 1 year ago

Does the updated park point apply to both heads the same? Or are there unique park points per head?

Edit: Saw the commit and I see it is per head. Perfect, thanks! I'll update my printer soon.

johncarlson21 commented 1 year ago

Yes.. there are technically 2 park points that can be used, but the pause feature uses one specific to the heads.

johncarlson21 commented 1 year ago

Closing this as this was updated in code. please update to the latest release or pre-release