-
Notifications
You must be signed in to change notification settings - Fork 31
PauseAtHeight: added extrude amount option #20
Conversation
See nallath#19 I have not tested this yet with an actual print
@@ -42,6 +42,14 @@ def getSettingDataString(self): | |||
"unit": "mm", | |||
"type": "float", | |||
"default_value": 0 | |||
}, | |||
"extrude_ammount": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo in the setting key 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just trying to be consistent with https://github.com/nallath/PostProcessingPlugin/blob/master/scripts/PauseAtHeight.py#L38 ;) No just kidding, I should probably fix both.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, right, didn't see that.
Fixing both would be nice, though it's not your obligation.
As usual I'm struggling with G-Code, it doesn't seem to extrude the amount I ask it to extrude, even after lowering the feedrate to a normal rate; The
Settings used for this G-Code
|
I think I see what you mean. The problem is that the material is retracted when you try to extrude 50. It'll extrude 40 then, because the filament was 10mm above the nozzle. What you want is to have the extrude one line lower, between |
Ah because it's at -10 (that 10mm above the nozzle)... Good catch. |
Same result, I see maybe 3mm being extruded with the settings mentioned above. |
The G-Code as it is generated now:
|
After doing another test I realize we'll also need to compensate for the extra extraction in the layers after the pause. |
Changing all the extrusion commands after the pause doesn't seem scalable, if anyone has an idea on how I could add extra extrusion without this that would be very welcome. |
@sjoerdtimmer ([email protected]) and I figured out how we could use G92 to reset the E position after the pause to the position it was before the pause.
@sjoerdtimmer ([email protected]) and I figured out how we could use G92 to reset the E position after the pause to the position it was before the pause. So this seems to work. It allows extra extruding, before a resume. It still works when someone doesn't change the material, it will just extrude more. What do you guys think? |
Could you remind me to look at this in a few days? Due to the release the 18th we're super busy right now. |
@nallath Here's a reminder. |
Thanks! Did you guys test both cases? The code looks fine, so i'm willing to merge it if it's tested. |
I totally understand, no problem, great work on the new Ultimaker. In the mean time I'm still improving this solution. Adhesion is a challenge, especially when using multiple material types, so I'm working on a redo layers setting. This would allow you to redo a number of layers after the pause. I'm also not sure how to explain that extrusion of 128 is needed, for Ultimaker2's. I'd like to recommend this in the description. |
I haven't tested this yet on anything else than a Ultimaker2. Simply resuming, without material change, works fine, it just extrudes extra. |
Extrudes extra? Won't that mess up the print? |
It's not perfect. It's extruded while the head is this in it's pause location, but it could be pulled towards the print. |
I've added a redo layers option to increase adhesion and I've improved the extrude amount description, including the seemingly random 128. |
So is it good to merge now? |
I think so, but I'm happy to improve it where needed. |
See #19
I have not tested this yet with an actual print