Skip to content

Fix FT_MOTION build with DISTINCT_E_FACTORS #27063

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

Merged

Conversation

vovodroid
Copy link
Contributor

@vovodroid vovodroid commented May 8, 2024

Fix undefined current_block in FTMotion when DISTINCT_E_FACTORS is enabled - add refer to stepper instance to access it.

@vovodroid vovodroid force-pushed the ft_motion_distinct_e_pr branch from 28b1b18 to c36cb50 Compare May 10, 2024 06:29
@vovodroid
Copy link
Contributor Author

@thisiskeithb, is it worth to merge this PR before fixing other FTM issues?

@thisiskeithb
Copy link
Member

@thisiskeithb, is it worth to merge this PR before fixing other FTM issues?

It's not up to me. I'm not involved with FT Motion-related code, so I don't know if any of the other FT Motion PRs will help or make things worse.

@vovodroid
Copy link
Contributor Author

vovodroid commented May 10, 2024

This PR is just about compiling. @narno2202, actually this code current_block->extruder is yours, this PR only adds stepper. to it.

@thinkyhead thinkyhead merged commit 4f3ebea into MarlinFirmware:bugfix-2.1.x May 17, 2024
@thisiskeithb thisiskeithb added this to the Version 2.1.3 milestone May 18, 2024
@vovodroid vovodroid deleted the ft_motion_distinct_e_pr branch May 18, 2024 07:28
EvilGremlin pushed a commit to EvilGremlin/Marlin that referenced this pull request May 15, 2025
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.

3 participants