Skip to content

Configure TMC interpolation per driver.#19828

Merged
thinkyhead merged 6 commits intoMarlinFirmware:bugfix-2.0.xfrom
ManuelMcLure:per_driver_interpolate
Oct 22, 2020
Merged

Configure TMC interpolation per driver.#19828
thinkyhead merged 6 commits intoMarlinFirmware:bugfix-2.0.xfrom
ManuelMcLure:per_driver_interpolate

Conversation

@ManuelMcLure
Copy link
Contributor

Description

Allow enabling/disabling TMC driver interpolation on a per-driver basis.

Benefits

Allows using interpolation on some drivers (such as X/Y) while disabling it on others (probably most useful for E).

Configurations

Configuration.zip

I have tested this with TMC 2209 drivers on a Re-ARM. It would be a good idea to have tests done on other TMC drivers before this is merged.

@thisiskeithb
Copy link
Contributor

You have the same “Use interpolation on X axis” comment for every axis.

@ManuelMcLure
Copy link
Contributor Author

You have the same “Use interpolation on X axis” comment for every axis.

Thanks for catching that!

@ManuelMcLure ManuelMcLure marked this pull request as draft October 21, 2020 05:08
@thinkyhead thinkyhead marked this pull request as ready for review October 21, 2020 20:05
@ManuelMcLure
Copy link
Contributor Author

ManuelMcLure commented Oct 22, 2020

@thinkyhead took care of the changes, and with using intpol() I'm less concerned about the change not working on other drivers, so I think this is ready to merge.

@thinkyhead thinkyhead merged commit 4fccb92 into MarlinFirmware:bugfix-2.0.x Oct 22, 2020
@ManuelMcLure ManuelMcLure deleted the per_driver_interpolate branch October 28, 2020 00:38
Speaka pushed a commit to Speaka/Marlin that referenced this pull request Nov 2, 2020
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
tharts pushed a commit to tharts/Marlin that referenced this pull request Jan 6, 2021
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
chrisjenda pushed a commit to chrisjenda/Marlin that referenced this pull request Apr 5, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 28, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments