Skip to content

Fix: Fixing the range check in ZAA code#13452

Merged
RF47 merged 1 commit into
OrcaSlicer:mainfrom
valerii-bokhan:zaa-range-check-fix
May 2, 2026
Merged

Fix: Fixing the range check in ZAA code#13452
RF47 merged 1 commit into
OrcaSlicer:mainfrom
valerii-bokhan:zaa-range-check-fix

Conversation

@valerii-bokhan

Copy link
Copy Markdown
Contributor

Description

A small fix for the range check in the ZAA code. Fixes this comment: #12736 (comment)

How to Download Pull Requests Artifacts for Testing

@github-actions

github-actions Bot commented May 2, 2026

Copy link
Copy Markdown

Hi @valerii-bokhan, you can manage the labels for this PR by /bot add-label and /bot remove-label

Allowed labels are:
bug-fix, enhancement, Localization, profile, QoL, UI/UX and dependencies.

Examples:

/bot add-label bug-fix
/bot add-label bug-fix, UI/UX
/bot remove-label bug-fix, UI/UX

@valerii-bokhan

Copy link
Copy Markdown
Contributor Author

/bot add-label bug-fix

@RF47 RF47 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@RF47 RF47 merged commit 6fd4984 into OrcaSlicer:main May 2, 2026
18 checks passed
dbaarda pushed a commit to dbaarda/OrcaSlicer that referenced this pull request May 3, 2026
* main: (381 commits)
  Fix: Fixing the range check in ZAA code (OrcaSlicer#13452)
  Fix role-based fan speeds being lost on layer change (OrcaSlicer#13390)
  PR OrcaSlicer#12525 follow-up (OrcaSlicer#13440)
  Correct center grid multiline (OrcaSlicer#13422)
  Fix to raft checking logic for slow down layers (OrcaSlicer#13415)
  Fix: Expose support_chamber_temp_control flag outside developer mode (OrcaSlicer#12924) (OrcaSlicer#13393)
  Tweak UI 1. Move ZAA options to comExpert 2. Expose zaa_dont_alternate_fill_direction to the UI 3. Remove ironing_expansion from UI as it's not implemented
  fixed an issue that zaa_enabled is redefined in both object and region. move all zaa config to region to keep it consistent
  Feature/per vendor update (OrcaSlicer#13394)
  Introducing Orca Cloud: https://cloud.orcaslicer.com (OrcaSlicer#13414)
  Hook up deps-windows.cmake too
  Add processor mapping for ARM64 -> DEPS_ARCH=arm64 With this change, builds on ARM64 hosts will successfully kick off! Unfortunately there are still missing/outdated dependencies, so the build doesn't yet succeed. This makes it much easier to check what we're failing on though.
  Fix outer wall for bridges (OrcaSlicer#13411)
  Fix and Improve 3MF impOrt (OrcaSlicer#13403)
  Fix a regression issue orca couldn't login into BBL cloud (OrcaSlicer#13407)
  revert pure format changes
  Fixed a bug where `zaa_dont_alternate_fill_direction` still had an effect even when ZAA is disabled.
  Fixed a bug where the scarf seam wouldn't work properly when ZAA was disabled.
  fix missing settings after merging
  Update profiles identation (OrcaSlicer#13375)
  ...
dbaarda pushed a commit to dbaarda/OrcaSlicer that referenced this pull request May 3, 2026
* main: (417 commits)
  Fix: Fixing the range check in ZAA code (OrcaSlicer#13452)
  Fix role-based fan speeds being lost on layer change (OrcaSlicer#13390)
  PR OrcaSlicer#12525 follow-up (OrcaSlicer#13440)
  Correct center grid multiline (OrcaSlicer#13422)
  Fix to raft checking logic for slow down layers (OrcaSlicer#13415)
  Fix: Expose support_chamber_temp_control flag outside developer mode (OrcaSlicer#12924) (OrcaSlicer#13393)
  Tweak UI 1. Move ZAA options to comExpert 2. Expose zaa_dont_alternate_fill_direction to the UI 3. Remove ironing_expansion from UI as it's not implemented
  fixed an issue that zaa_enabled is redefined in both object and region. move all zaa config to region to keep it consistent
  Feature/per vendor update (OrcaSlicer#13394)
  Introducing Orca Cloud: https://cloud.orcaslicer.com (OrcaSlicer#13414)
  Hook up deps-windows.cmake too
  Add processor mapping for ARM64 -> DEPS_ARCH=arm64 With this change, builds on ARM64 hosts will successfully kick off! Unfortunately there are still missing/outdated dependencies, so the build doesn't yet succeed. This makes it much easier to check what we're failing on though.
  Fix outer wall for bridges (OrcaSlicer#13411)
  Fix and Improve 3MF impOrt (OrcaSlicer#13403)
  Fix a regression issue orca couldn't login into BBL cloud (OrcaSlicer#13407)
  revert pure format changes
  Fixed a bug where `zaa_dont_alternate_fill_direction` still had an effect even when ZAA is disabled.
  Fixed a bug where the scarf seam wouldn't work properly when ZAA was disabled.
  fix missing settings after merging
  Update profiles identation (OrcaSlicer#13375)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants