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

Gcode viewer: Opening settings causes layers to vanish from model #3143

Closed
foosel opened this issue May 7, 2019 · 1 comment
Closed

Gcode viewer: Opening settings causes layers to vanish from model #3143

foosel opened this issue May 7, 2019 · 1 comment
Labels
bug Issue describes a bug done Done but not yet released
Milestone

Comments

@foosel
Copy link
Member

foosel commented May 7, 2019

Problem

When opening the settings while the gcode viewer is active, the model loses the first two layers. Opening settings again and again can make the whole model disappear from memory.

The bug can be seen by the viewer seeming to switch to a new layer after every opening of the settings, even though the viewer still claims to be at the first one.

The layers are also no longer sorted.

Steps:

  • Connect to virtual printer, select a file
  • switch to gcode viewer, let file load
  • open settings, close settings
  • note that a different layer is shown

JS Console with 22d7392 applied shows this on repeated openings:

Rerendering layer 0 of 16 with 1636 segments
packed_core.js?777f4219:9085 Rerendering layer 0 of 15 with 1125 segments
packed_core.js?777f4219:9085 Rerendering layer 0 of 14 with 1395 segments
packed_core.js?777f4219:9085 Rerendering layer 0 of 14 with 1395 segments
packed_core.js?777f4219:11018 [Violation] 'load' handler took 306ms
packed_core.js?777f4219:9085 Rerendering layer 0 of 13 with 1517 segments
packed_core.js?777f4219:9085 Rerendering layer 0 of 12 with 1162 segments
packed_core.js?777f4219:9085 Rerendering layer 0 of 12 with 1162 segments
packed_core.js?777f4219:11018 [Violation] 'load' handler took 172ms
packed_core.js?777f4219:9085 Rerendering layer 0 of 11 with 1160 segments
packed_core.js?777f4219:9085 Rerendering layer 0 of 10 with 1509 segments
packed_core.js?777f4219:9085 Rerendering layer 0 of 10 with 1509 segments

Notice the decreasing layer count.

@foosel foosel added the bug Issue describes a bug label May 7, 2019
@foosel foosel added this to the 1.3.12 milestone May 7, 2019
@foosel foosel self-assigned this May 7, 2019
@foosel
Copy link
Member Author

foosel commented May 7, 2019

Fixed by the above commit, pushed to maintenance, soon devel, to be part of 1.3.12.

@foosel foosel added the done Done but not yet released label May 7, 2019
@foosel foosel removed their assignment May 7, 2019
foosel added a commit that referenced this issue May 15, 2019
@foosel foosel closed this as completed in c1e8bc7 Oct 22, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue describes a bug done Done but not yet released
Projects
None yet
Development

No branches or pull requests

1 participant