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

Layer count starts at 1 instead of 0 #16

Closed
mstreurman opened this issue Sep 14, 2019 · 9 comments
Closed

Layer count starts at 1 instead of 0 #16

mstreurman opened this issue Sep 14, 2019 · 9 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@mstreurman
Copy link

Describe the bug the bug is.
When you start printing the layer count with the icon at the bottom left of the dashboard starts counting at 1 instead of 0 thus it always has 1 too many layers showing. The other plug in for it Displaylayerprogressplugin does show the correct count in the new window and on the printer

To Reproduce
Steps to reproduce the behavior:
Install dashboard 1.0
install Displaylayerprogressplugin
Slice object
Start printing
See the error

Expected behavior
To show the correct amount of layers

Screenshots
If applicable, add screenshots to help explain your problem.
N/A

Desktop (please complete the following information):
Intel Core i9 9900k
2x 8GB Corsair Vengeance LPX 3000MHz CL15
Asrock z390 Extreme4
nVidia GeForce RTX2080ti
250GB Samsung 960 EVO M.2 nVME
1TB Adata XPG SX8200 PRO M.2 nVME
Original Prusa i3 MK3S 3D-printer

Windows 10 Professional build 1903 running the latest updates for Windows and the latest drivers.
Edge browser (whatever is the latest version)

@StefanCohen
Copy link
Collaborator

Thanks for the feedback.

This was intentional. Starting at zero only makes sense from a development perspective but I don't think it makes sense for the user if the first layer is layer zero. I add 1 to the currentlayer and totallayers metrics from displaylayerprogress so for example a 10mm model sliced with .1mm layer height is reported as having a total of 100 layers (not 99) and the first layer is layer 1 (rather than 0).

I could add a user setting in the plugin to let the user choose what method to use if enough people want it but I would still choose the current method (start at 1) as default.

This is my opinion but feel free to change my mind.

@StefanCohen StefanCohen added the wontfix This will not be worked on label Sep 14, 2019
@mstreurman
Copy link
Author

wrong layer height
The problem here is that if you have the displaylayerprogress plugin installed you will see conflicting information. eg. everywhere it shows layer 69 but in the dashboard it shows layer 70. That makes for a very inconsistent experience . That is: unless you will drop the dependency on a different plug in and make your own implementation of it or if the user manually changes all the settings of the displaylayerprogress plug in.

@wizard04wsu
Copy link
Contributor

Perhaps there should be better distinction between completed layers and which layer is currently printing. Printing starts on layer 1, when 0 have been completed. Which number is the dashboard actually tracking?

@StefanCohen
Copy link
Collaborator

I’m not sure that l understand. The dashboard shows the total number of layers in the gcode and what layer is currently being printed. Completed layers is not displayed.

@wizard04wsu
Copy link
Contributor

That makes sense to me. And the tooltip does say "Current Layer".

@StefanCohen StefanCohen added enhancement New feature or request and removed wontfix This will not be worked on labels Sep 15, 2019
@StefanCohen StefanCohen removed this from To do in Dashboard Nextrelease Sep 15, 2019
@mkh595
Copy link

mkh595 commented Sep 16, 2019

Addition 1 should be not fixed in code but as parameter in pluging setup, e.g. like in 'DisplaLayerProgress Plugin', there is explanation why

Layer1

Model has 60 layers

Layer2

but in Dashboard are both values wrong due to fixed 'addition 1'

Layer3

@StefanCohen
Copy link
Collaborator

Thanks! I wasn't even aware of that settin in DisplayLayerProgress. I will remove the layer increment in the dashboard and update the README with the info.

@StefanCohen StefanCohen self-assigned this Sep 16, 2019
@StefanCohen
Copy link
Collaborator

Fixed in next release.

@StefanCohen StefanCohen added this to the 1.1.2 milestone Sep 16, 2019
@StefanCohen
Copy link
Collaborator

Fixed in 1.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants