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

Fix FT_MOTION build with DISTINCT_E_FACTORS #27063

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
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
62 checks passed
@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
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.

None yet

3 participants