Repository navigation
Remove settings OrcaSlicer ignores from the Sovol process profiles - #17085
Merged
SoftFever merged 1 commit intoOct 31, 2026
Merged
Conversation
wall_loop_direction and initial_solid_infill_acceleration in the three SV08 MAX processes, and minimum_sparse_infill_threshold in eight SV06 ACE and SV06 Plus ACE processes, are not options OrcaSlicer defines. The loader drops them, so removing them does not change the sliced output. The SV06 ACE files already set minimum_sparse_infill_area. Bump the Sovol bundle version to 02.04.00.09.
thalisantunes
temporarily deployed
to
merge-delegation
October 31, 2026 13:55 — with
GitHub Actions
Inactive
|
Hi @thalisantunes, you can manage the labels for this PR by Allowed labels are: Examples: |
thalisantunes
marked this pull request as ready for review
October 31, 2026 14:27
raistlin7447
pushed a commit
that referenced
this pull request
Oct 31, 2026
…17085) wall_loop_direction and initial_solid_infill_acceleration in the three SV08 MAX processes, and minimum_sparse_infill_threshold in eight SV06 ACE and SV06 Plus ACE processes, are not options OrcaSlicer defines. The loader drops them, so removing them does not change the sliced output. The SV06 ACE files already set minimum_sparse_infill_area. Bump the Sovol bundle version to 02.04.00.09. (cherry picked from commit 4274ea6)
SoftFever
pushed a commit
that referenced
this pull request
Nov 1, 2026
…17085) wall_loop_direction and initial_solid_infill_acceleration in the three SV08 MAX processes, and minimum_sparse_infill_threshold in eight SV06 ACE and SV06 Plus ACE processes, are not options OrcaSlicer defines. The loader drops them, so removing them does not change the sliced output. The SV06 ACE files already set minimum_sparse_infill_area. Bump the Sovol bundle version to 02.04.00.09. (cherry picked from commit 4274ea6)
This branch was previously deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Eleven Sovol process profiles carry settings that OrcaSlicer does not define. The loader ignores them, so they never affect a slice:
wall_loop_directionandinitial_solid_infill_accelerationin the three SV08 MAX processes (0.4, 0.6 and 0.8 nozzle).minimum_sparse_infill_thresholdin eight SV06 ACE and SV06 Plus ACE processes.Root cause: None of the three names exists in
src/.PrintConfigDef::handle_legacyclears a key it does not know, andConfigBase::set_deserialize_nothrowthen skips it without an error. The profile tool only flags a fixed list of retired keys, and these three are not on it. In all eight SV06 files the option that applies,minimum_sparse_infill_area, is already set to 10. The removed key said 0, so the effective value stays 10.Fix: I removed the 14 lines and raised
versioninSovol.jsonfrom02.04.00.08to02.04.00.09. No other key, value or key order changes.I did not rename
wall_loop_direction: clockwisetowall_direction: cw. The profile never applied it, and the default is counter clockwise, so a rename would change printed output. If clockwise walls are wanted on the SV08 MAX, that belongs in a separate change with print results.Screenshots/Recordings/Graphs
Not applicable. No UI change.
Tests
grep -rn -E "wall_loop_direction|initial_solid_infill_acceleration|minimum_sparse_infill_threshold" src/finds nothing.python3 scripts/orca_profile_tool.py checkreports 71 vendors, 0 files with errors and 0 files with warnings, both onmainand on this branch.upstream/mainand at this branch and compared the key/value pairs in order. The only differences are the 14 removed keys, plusversioninSovol.json. The remaining keys keep their values and their order.python3 scripts/orca_profile_tool.py version-check --base upstream/main:Sovol: 02.04.00.08 -> 02.04.00.09, 12 files changed.python3 -m unittest discover -s scripts/tests -t scripts: 381 tests, OK (1 skipped).scripts/check_profile.sh: all checks passed (profile_tool, version_bump, legacy_compat, validate_system, validate_slice, validate_filament_subtypes, validate_custom), including the validator binaries v2.4.0, v2.4.1 and v2.4.2.How to Download Pull Requests Artifacts for Testing