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

Walls printed one layer height higher when using semi-paralell printing #4004

Closed
Lecso11 opened this issue Dec 17, 2023 · 13 comments
Closed
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

@Lecso11
Copy link

Lecso11 commented Dec 17, 2023

What happened?

When printing multiple part with semi paralell printing, every part except the first has the walls printed one layer height higher (in my case 0.2mm) than it should be printed.
My first layer is 0.25mm, others 0.2. First part's first layer is perfect, but the other's walls are printed on 0.45mm height, then the infill on 0.25.
On the picture: left is first, rest are identically printed wrong.

Project file & How to reproduce

kampov3.zip
IMG_20231217_203811

Version

2.5.59.3

Operating system

windows 10

Printer model

Voron 2.4 klipper

@vovodroid
Copy link

I opened project, but it's configured to be printed by layer

image

What are your settings here?

@Lecso11
Copy link
Author

Lecso11 commented Dec 18, 2023

I opened project, but it's configured to be printed by layer

image

What are your settings here?

Hi!
Weird... Here they are:
image

@vovodroid
Copy link

My first layer is 0.25mm
0.3 actually - 75% of 0.4 nozzle.

others 0.2.
You have variable layer height, so it varies.

I don't see such issue. Second layer of all parts is printed at 0.505:

image

image

But actually despite putting efforts and time in this feature development, I'm now thinking that it's not necessary. I was initially thinking about stringing and printing time, but eventually stringing is not issue for me, and printing time also is almost the same. And it's better to print small objects by layer to allow better cooling.

@Lecso11
Copy link
Author

Lecso11 commented Dec 18, 2023

My first layer is 0.25mm
0.3 actually - 75% of 0.4 nozzle.

others 0.2.
You have variable layer height, so it varies.

I don't see such issue. Second layer of all parts is printed at 0.505:

image

image

But actually despite putting efforts and time in this feature development, I'm now thinking that it's not necessary. I was initially thinking about stringing and printing time, but eventually stringing is not issue for me, and printing time also is almost the same. And it's better to print small objects by layer to allow better cooling.

Edit: I don't know what happened, but you pretty much cannot see my settings. My first layer is 0.25, and while I have variable layer height, only for that sloping area, first few mms are 0.2mm layer height.
Not the second, the first layer's walls are printed on wrong height. But now that I thought about it, that 0.2mm higher, which results in the 0.45 might be my z-hop.
Also, second layer should be 0.25+0.2=0.45mm
See:
image

Actually this part of the infill is still too high:
image

But this is fine:
image

What you say is partly true, but I think it is still a very useful feature. While you might have no issue with stringing, there are materials that are pretty bad. And with middle-sized parts, or where cooling is not an issue, it still makes sense to use.
I think the feature has its use-case, I wanted this for a long time.

@vovodroid
Copy link

I don't know what happened, but you pretty much cannot see my settings.

I downloaded SS release, but didn't run config wizard, I guess all settings should be included in project file, isn't it?

there are materials that are pretty bad.

I've read about it. But I printed PLA, PETG, ASA, Nylon, Polycarbonate and TPU. TPU indeed is stringing. Other - nothing special.

@Lecso11
Copy link
Author

Lecso11 commented Dec 18, 2023

I don't know what happened, but you pretty much cannot see my settings.

I downloaded SS release, but didn't run config wizard, I guess all settings should be included in project file, isn't it?

there are materials that are pretty bad.

I've read about it. But I printed PLA, PETG, ASA, Nylon, Polycarbonate and TPU. TPU indeed is stringing. Other - nothing special.

Yes the project file should contain it.

Anyway, if you find the time looking into it, I would be thankful. I would like to use this feature.

@vovodroid
Copy link

vovodroid commented Dec 18, 2023

the project file should contain it.

So something strange here. I loaded your project and all setting are default:

image

@Lecso11
Copy link
Author

Lecso11 commented Dec 18, 2023

the project file should contain it.

So something strange here. I loaded your project and all setting are default:

image

kampov3forgithub.zip
try this one, It works for me. Loads everything.

@vovodroid
Copy link

They should be separated objects:
image

And full config is still not loaded:
image

Can you try open project in fresh SS installation, better on other machine?

@supermerill supermerill added the bug Something isn't working as intended label Dec 19, 2023
@supermerill
Copy link
Owner

note: i can reproduce, and can open your project on latest nightly.

@Lecso11
Copy link
Author

Lecso11 commented Dec 19, 2023

They should be separated objects: image

And full config is still not loaded: image

Can you try open project in fresh SS installation, better on other machine?

I am aware that instances are not handled, I forgot to split this one for you.

Merill could open it, I don't know what could be the issue with your slicer.

@supermerill
Copy link
Owner

Fixed.
That was a quite serious one.
The bug originates from the part that remove superfluous feedrate ( F3000). In this case, it's a wrong check: it has to check the line to remove the whole of it if it's only a G1 F3000, but not if it's G1 X1 Y2 F3000. The check was completely insufficient.

@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 Dec 19, 2023
@Lecso11
Copy link
Author

Lecso11 commented Dec 22, 2023

Fixed. That was a quite serious one. The bug originates from the part that remove superfluous feedrate ( F3000). In this case, it's a wrong check: it has to check the line to remove the whole of it if it's only a G1 F3000, but not if it's G1 X1 Y2 F3000. The check was completely insufficient.

Thank you!

@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 Dec 29, 2023
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

3 participants