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

Reset SV06 (Plus) ACE wall_sequence to default #8180

Closed
wants to merge 1 commit into from

Conversation

bcat
Copy link
Contributor

@bcat bcat commented Jan 25, 2025

@liberodark FYI in case you have thoughts on this

Description

The SV06 ACE profile Sovol originally supplied set "Wall printing order" to "Inner/Outer/Inner", and this got copied to the other layer height variants. But these profiles set "Wall loops" to 2, so "Inner/Outer/Inner" effectively becomes "Outer/Inner". This results in messy, quite noticeable Z seams.

It seems that "Inner/Outer" wall sequence is Orca's default, and IMO it results in better visual surface quality.

Screenshots/Recordings/Graphs

I'm not the best at photographing this, but here's the result of the current SV06 ACE profile behavior. Notice how jagged and gappy the seams on the left side of the print look:

Z seams with Inner/Outer/Inner wall order

And here's the same model printed with Inner/Outer wall sequence. The Z seams are still visible, but smoother and less eye catching:

s with Inner/Outer

Tests

Verified effects via some calibration prints.

@bcat
Copy link
Contributor Author

bcat commented Jan 25, 2025

On the subject of "other odd settings Sovol picked": thick bridges are on for these profiles, and I'm not sure that's an ideal default. I guess it's subjective, but they don't look great IMO. Needs some testing though.

@liberodark
Copy link
Contributor

liberodark commented Jan 25, 2025

Hi, @bcat

Thank for your tests.
I will look at what can be done in any case the removal of the option already seems to improve things but I would have to test it on my side too.
Before merge probably we can improve that.

PS : Note that this is an option that they have set as default.
"wall_sequence": "inner-outer-inner wall"

Also is probably better to change to : "wall_sequence": "outer wall/inner wall"
And I propose to establish a test protocol to validate this point.

  1. Use same benchmark calicat.stl.zip
  2. Use same configuration nozzle 0.4 at 0.20mm or 0.12mm (i think is better to test 0.20 or both).
  3. Try option in my side & in your side for compare here
  4. Validate best option here
  5. Merge

If you are ok with that ?

Best Regards

@liberodark
Copy link
Contributor

liberodark commented Jan 25, 2025

Test in 0.12 with inner-outer-inner wall :

image
image
image
image
image

@liberodark
Copy link
Contributor

Test in 0.12 with outer wall/inner wall :

image
image
image

@liberodark
Copy link
Contributor

liberodark commented Jan 25, 2025

Test in 0.12 with `` :
It's very difficult to see in the image but it's actually much prettier if we change the options.

image
image

@liberodark
Copy link
Contributor

liberodark commented Jan 25, 2025

Have make fix on my side : liberodark@1ee6069

Hi, @Noisyfox
What do you think about this ?
Better to remove or to set same as my fix ?

Best Regards

@bcat
Copy link
Contributor Author

bcat commented Jan 25, 2025

Thanks for the detailed investigation!

Regarding setting "wall_sequence": "inner wall/outer wall explicitly, this seems fine to me. I thought that was the default if the option was missing, but maybe I was mistaken. (For my own test prints, I just changed the setting to "Inner/Outer" in the GUI.)

Personally, in both your pics and mine I prefer changing to "Inner/Outer". From what I can tell, "Inner/Outer/Inner" makes the Z seam quite ugly (even more noticeable in real life than in pictures) when the profile defaults to only 2 walls, since it forces the seam to start on the outermost layer. So I think the current choice ("Inner/Outer/Inner") is just a small mistake/misunderstanding Sovol had in their original profile. :)

I have also noted almost none of the other printer profiles set "Inner/Outer/Inner". "Inner/Outer" seems to be more common (maybe for this reason?).

Have make fix on my side : liberodark@1ee6069

I am happy to pull in that change here (explicit set "wall_sequence": "inner wall/outer wall rather than just deleting the key) if you want. Or I can close this and you can send a separate PR. Zero preference here, I just want the defaults to look as nice as possible. :)

@liberodark
Copy link
Contributor

Have push my change thank for your work.

@bcat bcat closed this Jan 25, 2025
@bcat bcat deleted the sv06-ace branch January 25, 2025 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants