prusa3d / PrusaSlicer

G-code generator for 3D printers (RepRap, Makerbot, Ultimaker etc.)
https://www.prusa3d.com/prusaslicer/
GNU Affero General Public License v3.0
7.39k stars 1.87k forks source link

Disable sequential printing in vase mode on XL. #12632

Closed NelisW closed 3 weeks ago

NelisW commented 3 weeks ago

Description of the bug

Tried to print vase mode, multiple copies of the same part. I did not realise that the X-axis gantry will knock/squash parts already printed. For bed slingers the vertical sequential clearage makes sense, but not for coreXY: the bed moves and with it, all the parts. Prevent the user from making this naive mistake - disable sequential printing for XL, or at least warn the user.

Project file & How to reproduce

vase-mode-container-v2-short-XL.zip The attached 3mf file will print the small caps in front because they are low enough, but fail on the printing the tall containers at the back. In two of three test cases the bed actually titled by about 5-10 degrees, so this is a serious condition to be warned/disabled.

Checklist of files included above

Version of PrusaSlicer

2.7.4

Operating system

Windows 11

Printer model

XL 5TH

u89djt commented 3 weeks ago

(fellow user indicating an issue with your changed settings) Hi there, it looks like you've changed the clearance parameters such that the slicer thinks there will be sufficient clearance: image-1 Reset those to their default values and the slicer will not let you do it: image-2 This suggests that the hover text of the parameters isn't doing a great job of explaining how they're interpreted by the slicer. Re: differences between bed slingers and coreXY, what accelerates due to what forces is different, but the kinematics of the extruder and the X axis carriage in the reference frame of the bed and any objects on it are identical - your objects see exactly the same stuff happening directly above - so clearances work the same way.

NelisW commented 3 weeks ago

Hi Dave,

Thanks for replying so quickly!

I made the clearance values larger to be "more safe". So changes should not be the cause.

On second figure: I did not get this warning. But even so, the sideway clearance should be enough.

In any event, despite all above, the issue is not really the extruder clearance. Rather the issue is the objects placed anywhere where the x gantry can hit them. There is a similar issue in the MK3 and MK4, where the x-rods can knock or squash the part.

I think the safest route is to at least warn the user that vase mode and sequential print should not be used together.

Thanks for your trouble!

Nelis

On Mon, 22 Apr 2024, 17:56 Dave T., @.***> wrote:

(fellow user indicating an issue with your changed settings) Hi there, it looks like you've changed the clearance parameters such that the slicer thinks there will be sufficient clearance: image-1.png (view on web) https://github.com/prusa3d/PrusaSlicer/assets/75209005/c3e9f9ee-da7e-46b1-9047-260aab8e6222 Reset those to their default values and the slicer will not let you do it: image-2.png (view on web) https://github.com/prusa3d/PrusaSlicer/assets/75209005/8198b2a6-8e08-4b49-b519-286f7758a21a This suggests - and I would agree - that the hover text of the parameters is not doing its job.

— Reply to this email directly, view it on GitHub https://github.com/prusa3d/PrusaSlicer/issues/12632#issuecomment-2070000719, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABOR6SOQM2NVEVJRAHHXIL3Y6UXJDAVCNFSM6AAAAABGSXH4AKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZQGAYDANZRHE . You are receiving this because you authored the thread.Message ID: @.***>

u89djt commented 3 weeks ago

I'm learning a lot from the issues page - including that vase mode works in sequential mode now! I missed that :) Back to the specifics here: have another look at the hover text on the clearance values. Those numbers assert the radius of the extruder assembly, so a bigger value is safer, but the height asserts the height of the X axis carrier off/away from the plate. By setting that to 45mm, you've said there is 45mm of clearance under the carriage /available/. You were hoping to tell the slicer how much space you /need/, but that's not what it does. image-1 A bigger extruder radius is safer in many ways, but a bigger height entered into that box is definitely less safe. Vase mode won't affect this, I think?

u89djt commented 3 weeks ago

Running from the back of the plate to the front keeps the X axis carriage out of the way, I think? Would this work out well in real life? Don't trust my judgement here, I just edited this now. staggered back to front vase-mode-container-v2-short-XL.zip image image

NelisW commented 3 weeks ago

Hi Dave,

Thanks, understood. User error, as always with things Prusa!

I took this as clearance around the part to be printed.

Thanks for spending your time on this! I will close the bug report.

N

On Mon, 22 Apr 2024, 20:16 Dave T., @.***> wrote:

Yeah, have another look at the hover text on the clearance values. Those numbers assert the radius of the extruder assembly, so a bigger value is safer, but the height asserts the height of the X axis carrier off the plate. By setting that to 45mm, you've said there is 45mm of clearance under the carriage /available/. You were hoping to tell the slicer how much space you /need/, but that's not what it does. image-1.png (view on web) https://github.com/prusa3d/PrusaSlicer/assets/75209005/de467c87-079e-4a82-9a95-69ab97b24965 A bigger extruder radius is safer in many ways, but a bigger height entered into that box is definitely less safe.

— Reply to this email directly, view it on GitHub https://github.com/prusa3d/PrusaSlicer/issues/12632#issuecomment-2070513893, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABOR6SOAJNESIGO26LZFUALY6VHZFAVCNFSM6AAAAABGSXH4AKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZQGUYTGOBZGM . You are receiving this because you authored the thread.Message ID: @.***>

NelisW commented 3 weeks ago

Hi Dave

Saw these two mails with the zip just now. I will experiment with the two files and come back to you. I just have to get past this workday (starting work now in South Africa).

Regards

N

On Mon, 22 Apr 2024, 20:47 Dave T., @.***> wrote:

Running from the back of the plate to the front keeps the X axis carriage out of the way, I think? Would this work out well in real life? Don't trust my judgement here, I just edited this now. staggered back to front vase-mode-container-v2-short-XL.zip https://github.com/prusa3d/PrusaSlicer/files/15067411/staggered.back.to.front.vase-mode-container-v2-short-XL.zip image.png (view on web) https://github.com/prusa3d/PrusaSlicer/assets/75209005/e0efec5d-1a0c-4158-99f1-3ba221068488 image.png (view on web) https://github.com/prusa3d/PrusaSlicer/assets/75209005/13092dc2-ed5d-48b5-a767-2fa31389e2e2

— Reply to this email directly, view it on GitHub https://github.com/prusa3d/PrusaSlicer/issues/12632#issuecomment-2070621494, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABOR6SIWMLGCRR6H7HB7DITY6VLK7AVCNFSM6AAAAABGSXH4AKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZQGYZDCNBZGQ . You are receiving this because you authored the thread.Message ID: @.***>

u89djt commented 3 weeks ago

No worries! It sounds like we're both doing this for the love of the game. Frontier spirit and all that. I'm pretty sure the zip file in the comment I deleted will be bad news because it places the carriage on the side where there are parts already. I still think in MK3S :) The back to front ordering seems right when you slide the layer progress from nothing to a full plate imagining the carriage above it, so maybe that will work.

NelisW commented 3 weeks ago

Hi Dave,

I switched on the (nozzle?) travel lines, and then is appears to be safe. I can't test now, my solar battery is low (cloudy cold front passing over). As soon as I have full solar again I will print and let you know.

Thanks, yet again!

Nelis

On Tue, 23 Apr 2024, 09:16 Dave T., @.***> wrote:

No worries! It sounds like we're both doing this for the love of the game. Frontier spirit and all that. I'm pretty sure the zip file in the comment I deleted will be bad news because it places the carriage on the side where there are parts already. I still think in MK3S :) The back to front ordering seems right when you slide the layer progress from nothing to a full plate imagining the carriage above it, so maybe that will work.

— Reply to this email directly, view it on GitHub https://github.com/prusa3d/PrusaSlicer/issues/12632#issuecomment-2071595819, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABOR6SKGBIKZCPYBGLPRO7LY6YDELAVCNFSM6AAAAABGSXH4AKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZRGU4TKOBRHE . You are receiving this because you authored the thread.Message ID: @.***>

u89djt commented 3 weeks ago

What do you think the hover text for the height setting should say?