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

Top/Bottom Thickness error thickness if options Use adaptive layers enabled #7943

Closed
lk29 opened this issue Jun 16, 2020 · 7 comments
Closed
Labels
Status: Duplicate Duplicate of another issue. Type: Bug The code does not produce the intended behavior.

Comments

@lk29
Copy link

lk29 commented Jun 16, 2020

cura 4.6.1

Win 10/64

Use adaptive layers enabled

If the upper or lower shell coincides with the function of the adaptive layer height, the layer height in mm is not calculated correctly.

Probably a pre-calculated height calculation is used before the function is triggered.

@lk29 lk29 added the Type: Bug The code does not produce the intended behavior. label Jun 16, 2020
@mahtDFR
Copy link
Contributor

mahtDFR commented Jun 17, 2020

A project file or screenshot would help here, could you provide these please?

(File > Save... to save a project file)

@mahtDFR mahtDFR added the Status: Needs Info Needs more information before action can be taken. label Jun 17, 2020
@rburema rburema reopened this Jun 17, 2020
@rburema
Copy link
Member

rburema commented Jun 17, 2020

Sorry, accidentally closed. Wrong issue number in commit!

@lk29
Copy link
Author

lk29 commented Jun 17, 2020

Clipboard03
Clipboard02
Clipboard01

@lk29
Copy link
Author

lk29 commented Jun 17, 2020

@lk29
Copy link
Author

lk29 commented Jun 17, 2020

Layers 90-93 thickness 0.1 4*0.1 = 0.4
should be 0.8

@lk29
Copy link
Author

lk29 commented Jun 17, 2020

Clipboard05
Clipboard04

@rburema rburema added Status: Duplicate Duplicate of another issue. and removed Status: Needs Info Needs more information before action can be taken. labels Jun 18, 2020
@rburema
Copy link
Member

rburema commented Jun 18, 2020

Hi @lk29

Thanks for the info!

So it turns out this has been a known issue for a while, and is in fact a duplicate of #3913 we have an internal ticket for it, CURA-5104. Other related tickets are #3209 #4396 #5131 and #6188

This is basically the big issue preventing us from moving adaptive layer heights out of 'experimental', not only because it would require a refactor of the engine in some key parts (possibly the biggest issue), but also because it isn't clear then whether the number of layers or the thickness of the vertically specified setting should be leading (what if the user specifies number of layers vs. thickness of feature in mm. or vice versa, or both).

We hope to start working on this at some point though. It's a matter of prioritisation.

Since this is a duplicate however, and the original ticket is still open, please continue any discussion over there.

@rburema rburema closed this as completed Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Duplicate Duplicate of another issue. Type: Bug The code does not produce the intended behavior.
Projects
None yet
Development

No branches or pull requests

3 participants