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

Thick layers + thick raft + lightning infill = crash #9399

Closed
2 tasks
Pitel opened this issue Jan 17, 2023 · 2 comments
Closed
2 tasks

Thick layers + thick raft + lightning infill = crash #9399

Pitel opened this issue Jan 17, 2023 · 2 comments

Comments

@Pitel
Copy link

Pitel commented Jan 17, 2023 •
edited
Loading

Description of the bug

Crash. I expected it to, um, well, not crash.

Project file & How to reproduce

  1. Have Prusa MINI with 0,6 nozzle profile
  2. Add default cube (or probably any other STL or anything)
  3. Choose any layer height larger than 0.2
  4. Set infill type to Ligtning
  5. Set raft to 5 or more layers
  6. Crash 💥

You can do steps 3-5 in any order, but after the last step, it always crashes.

Checklist of files included above

  • Project file
  • Screenshot

Version of PrusaSlicer

Verze 2.5.0+linux-x64-GTK3

Operating system

Ubuntu 22.10

Printer model

MINI
@Pitel Pitel changed the title 0.4 layer + 5 layers of raft + lightning infill = crash thick layer + 5 layers of raft + lightning infill = crash Jan 17, 2023
@Pitel Pitel changed the title thick layer + 5 layers of raft + lightning infill = crash Thick layer + 5 layers of raft + lightning infill = crash Jan 17, 2023
@Pitel Pitel changed the title Thick layer + 5 layers of raft + lightning infill = crash Thick layers + thick raft + lightning infill = crash Jan 17, 2023
@rtyr
Copy link
Collaborator

rtyr commented Jan 17, 2023

Reproduced, thanks.

Lightning-Box.zip

@bubnikv
Copy link
Collaborator

bubnikv commented Jan 17, 2023

Fixed with 402affb
It will be fixed in first PrusaSlicer 2.6.0 public alpha.

@bubnikv bubnikv closed this as completed Jan 17, 2023
bubnikv added a commit that referenced this issue Jan 31, 2023
@bubnikv
Fix of #9399 Thick layers + thick raft + lightning infill = crash
402affb 
FillLightning::Filler infill wrapper was not aware of the fact
that Layer::id() returns layer index that includes raft layers.
This is newly handled by the FillLightning::Filler wrapper.
lukasmatena added a commit that referenced this issue Jun 9, 2023
@lukasmatena
Revert "Fix of #9399 Thick layers + thick raft + lightning infill = c…
f044a11 
…rash"

This reverts commit 402affb.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Pitel @bubnikv @rtyr