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

Variable layer height not respecting top thickness #3293

Open
badvision opened this issue Feb 12, 2018 · 11 comments
Open

Variable layer height not respecting top thickness #3293

badvision opened this issue Feb 12, 2018 · 11 comments
Labels
Type: Improvement Improvement to existing functionality.

Comments

@badvision
Copy link

Loving the new variable layer height feature but I'm printing several pieces at once and some parts are getting pillowing because I think the layers at that part were thin and it picked the same number of layer it would have otherwise picked, basically there wasn't enough material extruded in those layers to cover the infill correctly but I get good results in other parts of the print. Should this feature instead just go off of top /bottom thickness and ignore # of layers?

@jackha
Copy link
Contributor

jackha commented Feb 13, 2018

Can you show a case where it didn't work out as you hoped? I think especially for round like parts the top and bottom are important.

@ianpaschal ianpaschal added the Status: Needs Info Needs more information before action can be taken. label Feb 14, 2018
@ianpaschal
Copy link
Contributor

Reporter has not provided more information for more than 2 weeks, so closing.

@Ghostkeeper
Copy link
Collaborator

Ghostkeeper commented Mar 16, 2018

This still reproduces. It's actually quite clear. Here's what happens:

  • The user specifies that he wants a top thickness of 1mm.
  • Cura (front-end) calculates based on the layer height that this means that the user wants 10 top layers.
  • Because of Adaptive Layer Height, CuraEngine changes the layer height in this zone to 0.04mm.
  • CuraEngine draws 10 top layers at 0.04mm.
  • The top thickness becomes 0.4mm instead of 1mm.

This is in my opinion one of the most serious issues with adaptive layer height because the feature quite often makes the top layers much smaller than the normal layer height. It's also kind of hard to find a good solution for this.

@Ghostkeeper Ghostkeeper reopened this Mar 16, 2018
@Ghostkeeper Ghostkeeper added Type: Bug The code does not produce the intended behavior. Category: CuraEngine and removed Status: Needs Info Needs more information before action can be taken. labels Mar 16, 2018
@Ghostkeeper
Copy link
Collaborator

I'm thinking that maybe we should disable the "Top Layers" setting that allows the user to specify the number of layers as a count, when adaptive layers is enabled, and then use the specified height instead, so that you get more top layers in areas where the layer height is smaller.

@treymd
Copy link

treymd commented Apr 30, 2018

I was about to report this bug as well, but it is already here. I have my top thickness set to 1.6mm and adaptive layers turned on. Cura ends up printing 8 layers at 0.1 layer height instead of the 0.2 default the print is set at, cutting the thickness of the top layer in half. Just confirming that this is indeed happening.

@mechaMecha
Copy link

Experiencing the exact same problem. Definitely a bug. For now I need to double or triple the top thickness setting to get acceptable top layers.

@mechaMecha
Copy link

mechaMecha commented May 20, 2018

This isn't even a difficult bug to reproduce. All you need to do is take a cube, fillet the top surface, then slice it in Cura with adaptive layers turned on. Slice it using .15 mm as the base layer height, 0.1 as the maximum variation, and top thickness set to 1 mm. There will be 7 top layers generated, and each layer will be approximately .05 mm thick. That results in a top surface that is approximately .35 mm thick, not anywhere close to the specified 1 mm.

@ChrisTerBeke ChrisTerBeke added Type: Improvement Improvement to existing functionality. and removed Type: Bug The code does not produce the intended behavior. labels Jun 10, 2018
@SmileyNeil
Copy link

SmileyNeil commented May 16, 2021

Same issue applies to bottom layers.

Example project attached showing issue with top layers by printing a cube flat on the bed and another angled slightly off level:

Layer height is set to 0.2mm, top/bottom thickness set to 1mm -> 5 top/bottom layers. Adaptive variation and step size are set to 0.1mm (though the effect shows up for other values).
The effect of the bug is that the top/bottom layer count (5) is respected instead of the top/bottom thickness, so enabling adaptive layers causes an adaptive top/bottom thickness.
On the cube printed flat, the top has 5 top layers that are 0.3mm thick -> 1.5mm thick top.
On the cube printed slightly tilted the top has 5 top layers that are 0.1mm thick -> ~0.5mm thick top.

adaptiveLayerTopBottomThickness.zip

@Ghostkeeper
Copy link
Collaborator

As for an update, this issue is still in our backlog. Developers, refer to issue CURA-5104.

It is considered a requirement for moving Adaptive Layers out of experimental.

@smartavionics
Copy link
Contributor

See https://github.com/smartavionics/Cura/releases for a Cura variant that implements what is being asked for here (i.e. the thicknesses of the top and bottom skins are maintained when variable layer heights are enabled).

@timperlabs
Copy link

This is still happening in Cura 4.13.

It might be useful to change the popover tip for this feature to include known limitations so that new users don't have to spend much time re-discovering why their rounded top surfaces have holes or irregularities.

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

No branches or pull requests