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.

Sign up for GitHub

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

Jump to bottom

Support Dense layer on [Automatic] does not work as intended #1657

Closed
nfons opened this issue Oct 18, 2021 · 2 comments
Closed

Support Dense layer on [Automatic] does not work as intended #1657

nfons opened this issue Oct 18, 2021 · 2 comments
Labels
bug Something isn't working as intended fix is live in the last release Please download /build the last release and try to reproduce.

Comments

@nfons
Copy link

nfons commented Oct 18, 2021

Describe the bug
On latest slicer Automatic should put a thin bridge from perimeter to perimeter...however on certain models, it is not doing this, but having floating bridges.

Base model:
image

image

but the next layer:

image

attached zip

>> Project File <<
Please upload a ZIP archive containing the project file used when the problem arise. Please export it just before the problem occur. Even if did nothing and there is no object, export it as it contains your current configuration.

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.
TEST.zip

Desktop (please complete the following information):

  • OS: [e.g. MacOS, Linux, Windows]
  • Version [e.g. 2.3.55.5], Use Help->About_

Additional context
Add any other context about the problem here.
@supermerill supermerill added bug Something isn't working as intended fixed for the next version That means that you should be able to test it in the latest nightly build labels Oct 18, 2021
supermerill added a commit that referenced this issue Oct 19, 2021
@supermerill
dense_infill (automatic) rework
efe1824 
 * better coverage
 * now print in the right sequence
 * fixed overlap (20% of bridge width) with everything for a good grip
#1657
supermerill added a commit that referenced this issue Oct 22, 2021
@supermerill
dense_infill (automatic) rework
58c73b1 
 * better coverage
 * now print in the right sequence
 * fixed overlap (20% of bridge width) with everything for a good grip
#1657
@supermerill supermerill added fix is live in the last release Please download /build the last release and try to reproduce. fixed for the next version That means that you should be able to test it in the latest nightly build and removed fixed for the next version That means that you should be able to test it in the latest nightly build fix is live in the last release Please download /build the last release and try to reproduce. labels Oct 23, 2021
@nfons
Copy link
Author

nfons commented Oct 27, 2021

still in latest (2.3.57.3)

image

@nfons
Copy link
Author

nfons commented Oct 27, 2021 •
edited

project zip: ERROR.zip

@supermerill supermerill added fixed for the next version That means that you should be able to test it in the latest nightly build and removed fix is live in the last release Please download /build the last release and try to reproduce. labels Oct 27, 2021
supermerill added a commit that referenced this issue Oct 27, 2021
@supermerill
Don't allow holes in infill_dense
db8d152 
Ensure that infill_every_layers and infill_only_where_needed when infill_dense,
  because it can't tolerate voids yet (they create different regions)
  TODO: find a way to merge sparse & voids and "recover" from that merge.
no sparse + dense infill >better_than> infill_only_where_needed + dense infill
#1657
@supermerill supermerill added fix is live in the last release Please download /build the last release and try to reproduce. and removed fixed for the next version That means that you should be able to test it in the latest nightly build labels Nov 7, 2021
@supermerill supermerill mentioned this issue Mar 30, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working as intended fix is live in the last release Please download /build the last release and try to reproduce.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nfons @supermerill