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

Adding custom move feedrate for G26 Travel moves, Original #20729 #20879

Merged
merged 6 commits into from
Jan 26, 2021

Conversation

ScrewThisBanana
Copy link
Contributor

THIS PULL REQEUST IS A DUPLICATE OF #20729 AS SOMETHING "WENT WRONG" / GOT REVERTED DURING THE MERGE

Requirements

G26_MESH_VALIDATION enabled in configuration, no additional requirements

Description

This commit adds an additional configuration parameter (G26_XY_FEEDRATE_MOVE) in the Configuration.h file that can be used to specify the movement speed during the G26 validation pattern command during moves without extrusion.

The added feature was tested on my personal printer (TwoTrees Sapphire Pro) and works like a charm. Only x/y movements without e components take the newly introduced feedrate. All other behaviours of the G26 command stay the same.

Benefits

The amount of time used for G26 is drastically reduced as the feedrate for print moves is now different from moves without E-components

Configurations

platformio.ini.txt
Configuration.h.txt

Related Issues

Closes #20615

ScrewThisBanana added 6 commits December 31, 2020 17:47
This commit adds an additional configuration parameter that can be used to specify the movement speed during the G26 validation pattern command during moves without extrusion.

Closes MarlinFirmware#20615
This commit adds a default 'G26_XY_FEEDRATE_MOVE' value (max movement speed / 1.5) in the G26.cpp - same behaviour as the default 'G26_XY_FEEDRATE' value
Configuration parameter renamed for better readability and consistency

MarlinFirmware#20615
Changed default value for 'G26_XY_FEEDRATE_TRAVEL' from 150 mm/s to 100 mm/s for safety purposes, comment alignment

MarlinFirmware#20615
As stated in issue MarlinFirmware#20615 and the matching PR, the speed values for G26 are aligned correctly.
@ScrewThisBanana ScrewThisBanana changed the title Bugfix 2.0.x Adding custom move feedrate for G26, Original #20729 Jan 25, 2021
@ScrewThisBanana ScrewThisBanana changed the title Adding custom move feedrate for G26, Original #20729 Adding custom move feedrate for G26 Travel moves, Original #20729 Jan 25, 2021
@Roxy-3D
Copy link
Member

Roxy-3D commented Jan 26, 2021

Thank You @ScrewThisBanana ! I'm eager to try this out. I'm not sure how to 'correctly' format that very complicated ternary code at line 237-240. But what you have makes it as easy to understand as possible.

@Roxy-3D Roxy-3D merged commit 7f4c5b8 into MarlinFirmware:bugfix-2.0.x Jan 26, 2021
thinkyhead added a commit to MarlinFirmware/Configurations that referenced this pull request Jan 26, 2021
@Roxy-3D
Copy link
Member

Roxy-3D commented Jan 26, 2021

@thinkyhead, The example Configuration.h files are handled automatically now, right?

Jyers pushed a commit to Jyers/Marlin that referenced this pull request Feb 3, 2021
…ware#20729 (MarlinFirmware#20879)

* Adding custom move feedrate for G26

This commit adds an additional configuration parameter that can be used to specify the movement speed during the G26 validation pattern command during moves without extrusion.

Closes MarlinFirmware#20615
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
…ware#20729 (MarlinFirmware#20879)

* Adding custom move feedrate for G26

This commit adds an additional configuration parameter that can be used to specify the movement speed during the G26 validation pattern command during moves without extrusion.

Closes MarlinFirmware#20615
zillarob pushed a commit to zillarob/Marlin that referenced this pull request Feb 25, 2021
…ware#20729 (MarlinFirmware#20879)

* Adding custom move feedrate for G26

This commit adds an additional configuration parameter that can be used to specify the movement speed during the G26 validation pattern command during moves without extrusion.

Closes MarlinFirmware#20615
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
…ware#20729 (MarlinFirmware#20879)

* Adding custom move feedrate for G26

This commit adds an additional configuration parameter that can be used to specify the movement speed during the G26 validation pattern command during moves without extrusion.

Closes MarlinFirmware#20615
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 29, 2021
…ware#20729 (MarlinFirmware#20879)

* Adding custom move feedrate for G26

This commit adds an additional configuration parameter that can be used to specify the movement speed during the G26 validation pattern command during moves without extrusion.

Closes MarlinFirmware#20615
thinkyhead pushed a commit that referenced this pull request Apr 30, 2021
…0879)

* Adding custom move feedrate for G26

This commit adds an additional configuration parameter that can be used to specify the movement speed during the G26 validation pattern command during moves without extrusion.

Closes #20615
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