Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce option to control one wall draft shield #8562

Conversation

igiannakas
Copy link
Contributor

@igiannakas igiannakas commented Feb 25, 2025

Description

Introduce option to control whether one wall or all skirt walls are extruded from the second layer onwards.

Fixes #7424

Screenshots/Recordings/Graphs

(edit - screenshots below updated following latest code commit after Softfever's feedback).

image

image

image

image

Tests

Tested with option enabled & disabled
Tested with multi color print
Tested with draft shield on/off

@igiannakas igiannakas changed the title Introduce option to control number of skirt walls after first layer Introduce option to control whether one or multiple skirt walls after first layer are extruded Feb 25, 2025
@SoftFever
Copy link
Owner

SoftFever commented Feb 26, 2025

Hmmm
I'm thinking of reducing confusion and avoid showing too many options for people who don’t use the draft shield at all (which is probably 99.9% of users TBH)
I suggest we make following changes instead, please let me what do you think.

  1. The skirt will respect the skirt height unless the draft shield option is enabled.
  2. If the draft shield is enabled, Orca will disable the skirt height option and display a new option: 'Single loop draft shield.' If this option is selected, Orca will ignore skirt loops for layers beyond the first and create a single loop."

@igiannakas
Copy link
Contributor Author

Makes sense I think - so basically always print a full fat, all perimeter skirt and if a draft shield is selected, disable the skirt height option (as it does today) and display the option to have a single or multiple wall draft shield, if I got you right?

@SoftFever
Copy link
Owner

Makes sense I think - so basically always print a full fat, all perimeter skirt and if a draft shield is selected, disable the skirt height option (as it does today) and display the option to have a single or multiple wall draft shield, if I got you right?

Exactly!

@tetsu97
Copy link

tetsu97 commented Feb 28, 2025

Any chance this will be resolved in the final version 2.3 please?

@igiannakas
Copy link
Contributor Author

I’ll try to find some time over the weekend to address softfever’s feedback and hopefully this could make it into the next beta.

@igiannakas
Copy link
Contributor Author

igiannakas commented Mar 5, 2025

@SoftFever Done and also some small cleanup on what fields are enabled disabled for the skirt loops = 0 scenario.

image

image

image

image

@igiannakas igiannakas changed the title Introduce option to control whether one or multiple skirt walls after first layer are extruded Introduce option to control one wall draft shield Mar 5, 2025
@igiannakas
Copy link
Contributor Author

Final update - renamed option to "Single loop draft shield" through the code base as per Softfever's feedback.

Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.
Thank you so much!

@SoftFever SoftFever merged commit ec213e9 into SoftFever:main Mar 8, 2025
16 checks passed
@igiannakas igiannakas deleted the Introduce-option-to-control-number-of-walls-on-skirt-after-first-layer branch March 8, 2025 08:58
@vovodroid
Copy link
Contributor

To be honest I don't really understand why use several loops-layers skirt. For easy removing one loop for extra layer(s) is good enough, like this:

image

@igiannakas
Copy link
Contributor Author

There have been several reports that the single wall on the second layer was not enough to enable easy detachment of the skirt from the plate. More walls make the second layer more robust to ease detachment.

as the material waste is minimal (most skirts are two / three layers tall tops) it makes sense from a UI and ease of use perspective to just default it to the lowest common denominator which is multiple loops for skirts.

For draft shields there is the option available to choose a more robust shield or a less dense one, as material use there is more significant.

@vovodroid
Copy link
Contributor

vovodroid commented Mar 8, 2025

Yes, I remember, but from my experience even one loop in second layer is enough to use it like "handle" to pull skirt off the bed. May be it's bad/material depending.

@SoftFever
Copy link
Owner

It also depends on the nozzle, I guess. In the case of a 0.2 mm nozzle, a single loop on the second layer might not be enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Skirt not maintaining loop count across layers in recent updates.
4 participants