Skip to content

CURA-13050 Use the correct speed reference when initiating a Z only move to the transient height#2311

Merged
HellAholic merged 1 commit into5.13from
CURA-13050
Mar 20, 2026
Merged

CURA-13050 Use the correct speed reference when initiating a Z only move to the transient height#2311
HellAholic merged 1 commit into5.13from
CURA-13050

Conversation

@HellAholic
Copy link
Contributor

The travels between the models and in finalize() in "one at a time" to transient height are in Z axis only and it should use the correct speed for z (speed_z_hop) that rather than the travel speed for XY (speed_travel).
The FW should already filter the speeds based on max Z feedrate but that does not mean we can set an incorrect value for pure Z axis travels.

CURA-13050

Resolves:
Ultimaker/Cura#21481

The travels are in Z axis only and it should use the correct speed for that rather than the travel speed for XY
@HellAholic HellAholic changed the title Use speed_z_hop for z-hop travel moves CURA-13050 Use the correct speed reference when initiating a Z only move to the transient height Mar 19, 2026
@HellAholic HellAholic added PR: Community Contribution 👑 Community Contribution PR's PR: BooBoos 😇 PR: BooBoos 😇 Suggestions for Typos, like the NoMesh or there is a suggested refactor labels Mar 19, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2026

Test Results

30 tests  +30   30 ✅ +30   4s ⏱️ +4s
 1 suites + 1    0 💤 ± 0 
 1 files   + 1    0 ❌ ± 0 

Results for commit f00e65d. ± Comparison against base commit f10789d.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@wawanbreton wawanbreton left a comment

Choose a reason for hiding this comment

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

Makes sense, those travels are only vertical indeed so speed_z_hop fits better here

@HellAholic HellAholic merged commit f5d2b37 into 5.13 Mar 20, 2026
25 checks passed
@HellAholic HellAholic deleted the CURA-13050 branch March 20, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: BooBoos 😇 PR: BooBoos 😇 Suggestions for Typos, like the NoMesh or there is a suggested refactor PR: Community Contribution 👑 Community Contribution PR's

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants