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

[Feature request]Z full step/mm setting #239

Closed
liftbag opened this issue May 12, 2020 · 11 comments
Closed

[Feature request]Z full step/mm setting #239

liftbag opened this issue May 12, 2020 · 11 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

@liftbag
Copy link

liftbag commented May 12, 2020

Version

2.2.50+

Operating system type + version

any

3D printer brand / version + firmware version (if known)

any

Request

  • I just noticed the most recent version of Slic3r has an interesting setting, the Z full step/mm setting
    I've always wondered what level of precision can guarantee the management of continuously variable layers, where the Z axis remains for an entire layer in intermediate positions between the full steps, or in the layers where the supports are generated.
    I believe that Ranellucci's approach (or jenkins?) to the problem is substantially correct.
    I wonder if it was possible to port this optional and experimental setting also on slic3r++.

The version I have reviewed is the slic3r-1.3.1-dev-54a31eed of 2020-01-27, available here:
https://dl.slic3r.org/dev/win/

Schermata 2020-05-12 alle 18 56 07

Is this a new feature request?
Yes

@supermerill
Copy link
Owner

supermerill commented May 12, 2020

I just tough of that last week
#219
but it's a bit complicated to implement it well.
Won't be used for vase mode, obviously.

@liftbag
Copy link
Author

liftbag commented May 12, 2020

Won't be used for vase mode, obviously.

Of course :)

@supermerill supermerill added this to the v52 milestone Jun 3, 2020
supermerill added a commit that referenced this issue Jun 10, 2020
seems to work without breaking.
todo: change all slicing stuff from double to scaled int.
supermerill added a commit that referenced this issue Jun 10, 2020
seems to work without breaking.
todo: change all slicing stuff from double to scaled int.
@supermerill supermerill added the fixed for the next version That means that you should be able to test it in the latest nightly build label Jun 10, 2020
@supermerill
Copy link
Owner

was more work than i though

@liftbag
Copy link
Author

liftbag commented Jun 10, 2020

was more work than i though

Sorry, I'm a noob about programming. I thought it was possible to bring it directly from slic3r.

@supermerill
Copy link
Owner

supermerill commented Jun 10, 2020

the slic3r implementation is extremely naive (or efficient). And now, prusa added other way to compute layers.

So I use it to round all layers computations and ensure that the manual, the automatic and the standard layers computation (+ supports) are all making z with a multiple fraction of it.

supermerill added a commit that referenced this issue Jun 10, 2020
seems to work without breaking.
todo: change all slicing stuff from double to scaled int.
supermerill added a commit that referenced this issue Jun 10, 2020
seems to work without breaking.
todo: change all slicing stuff from double to scaled int.
supermerill added a commit that referenced this issue Jun 11, 2020
seems to work without breaking.
todo: change all slicing stuff from double to scaled int.
@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 Jun 24, 2020
@Czarnyszarf
Copy link

What parameters should I use when I don't know the Z steps for my printer?

Can I disable it?

@supermerill
Copy link
Owner

supermerill commented Jun 24, 2020

put something low like the default one.
The gcode won't output a value lower than 0.001 anyway, so it's like disabling it.
I'll change the tooltip to explain that.

@Czarnyszarf
Copy link

by the way, setting it as 0 crashes the program :)

@supermerill
Copy link
Owner

erf, i didn't thought of it. Of course, there may be a division by 0...

@supermerill supermerill added bug Something isn't working as intended and removed enhancement fix is live in the last release Please download /build the last release and try to reproduce. labels Jun 24, 2020
supermerill pushed a commit that referenced this issue Jun 29, 2020
also change default for min_width_top_surface
@supermerill supermerill added the fixed for the next version That means that you should be able to test it in the latest nightly build label Jun 29, 2020
@supermerill supermerill removed this from the v52 milestone Jun 29, 2020
supermerill pushed a commit that referenced this issue Jul 31, 2020
Also increase precision from 3 to 6 for z if using z_step, because it may matters now.
also change default for min_width_top_surface
@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 Aug 24, 2020
@Photogad
Copy link

What setting should I use for a stock Prusa Mk3S?

@supermerill
Copy link
Owner

As there is a Mesh bed leveling, the z stepper are never at a fulls tep, so if you want to use this field you need to know the microstep.

based on https://forum.prusaprinters.org/forum/original-prusa-i3-mk3s-mk3-hardware-firmware-and-software-help/motor-step-angle-and-screw-pitch-for-mk3-1.8-and-.8mm/, 0.0025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

No branches or pull requests

4 participants