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

[ 2.6-alpha5 ] Provide a way to disable the Vertical Shell thickness completely #10074

Closed
2 tasks
meltiseugen opened this issue Mar 15, 2023 · 8 comments
Closed
2 tasks

Comments

@meltiseugen
Copy link

Description of the bug

Now vertical shell thickness is on by default and can not be turned off.

Project file & How to reproduce

Have the checkbox to control the vertical shell thickness solid infill back

Checklist of files included above

  • Project file
  • Screenshot

Version of PrusaSlicer

2.6-alpha5

Operating system

macOS

Printer model

Anycubic Mega X

@sarusani
Copy link

"The old "Ensure vertical shell thickness" parameter used to switch between the upstream Slic3r and Slic3r Prusa Edition algorithms to ensure vertical wall thickness. The new algorithm implemented in PrusaSlicer 2.6.0-alpha5 is significantly better than either of the two old algorithms; thus, the switch "Ensure vertical shell thickness" was removed."

@meltiseugen
Copy link
Author

"The old "Ensure vertical shell thickness" parameter used to switch between the upstream Slic3r and Slic3r Prusa Edition algorithms to ensure vertical wall thickness. The new algorithm implemented in PrusaSlicer 2.6.0-alpha5 is significantly better than either of the two old algorithms; thus, the switch "Ensure vertical shell thickness" was removed."

Yes, but what if I don't want to have those extra solid infills added in the slice?
Basically I just want an option to turn that on/off

@sarusani
Copy link

There was never an option to turn them off. The old box just switched between two different algorithms to calculate them. This is no longer needed since the new algorithm replaces both of them.

@Snuff1eupagus
Copy link

@meltiseugen

I agree with you, regardless of what ticking the radio button did in the past, the result\affect was it removed many areas of ensure vertical shell thickness. while there are workarounds that can help with this, it's proven very inconvenient to have to do compared to just being able to turn them off. There should be a button to remove them when unwanted. I would also say in this same paragraph that I'd like to have the control to switch back to the older bridging method, and seams from 2.4.2 when I need to.

I don't understand why it has to be one way or the other, why can't we have both. Now we have "ensure vertical shell" that we have to jump through hoops to get rid of if we don't want/need it, Internal bridging that you can't control if/when we need to, and Seams that won't do anything we want them to. How are these improvements? when all your doing is taking the control from us and then saying to us "oh what did you need that for?"

@Tupson444
Copy link

I also agree that we should be able to completely turn off the "ensure vertical shell thickness", whatever it is called and whatever algorithm is used. There should simply be an option to use none (with a proper tooltip explaining why it's usually needed).

And also, on the other hand, how it is decided what is "better"? If it's only based on reports how the old algorithm doesn't work correctly, it's easy to miss that many people might be using it and simply not having a reason to complain because it works as they want it to work. Of course things should be changed and improved, but I agree with @Snuff1eupagus that we should have a choice.

An idea: if something becomes obsolete and is being replaced with a new feature/algorithm, don't remove it for some time, but instead actively ask users at installation of a new version to give feedback if that should really be removed or not. Users would then have enough time (at least until the next release) to test both old and the new algorithms and compare, then based on user feedback, decide if there is still a reason to keep the old one or if nobody cares and it should definitely be removed.

@meltiseugen
Copy link
Author

I also agree, we need a way to disable this entirely.
I will change the issue title to reflect this accordingly.

@meltiseugen meltiseugen changed the title 2.6 alpha: bring back checkbox for ensure vertical shell thickness [ 2.6-alpha5 ] Provide a way to disable the Vertical Shell thickness completely Mar 17, 2023
@AudreyAP
Copy link

AudreyAP commented Jun 29, 2023

I can't seem to find a way to do this in 2.6 - not sure why this issue was closed. I cannot figure out how to get rid of this extra infill. It literally doubles print time when compared to 2.5.

Benchy example:

image

@Tupson444
Copy link

This issue is closed because another one is open: #10102

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

No branches or pull requests

5 participants