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

Remove incomplete dual stepper configuration #4824

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

gudnimg
Copy link
Collaborator

@gudnimg gudnimg commented Dec 21, 2024

If either Y_DUAL_STEPPER_DRIVERS or Z_DUAL_STEPPER_DRIVERS is defined the compilation will fail with an error message. This configuration will likely never be fully implemented unless someone from the community steps up.

I was looking into cleaning up the configuration headers in general, and thought it would be best to do this removal separately since its a lot more code.

@gudnimg gudnimg requested review from leptun and 3d-gussner December 21, 2024 14:08
@gudnimg gudnimg changed the title Dual stepper setup removal Remove incomplete dual stepper configuration Dec 21, 2024
@gudnimg gudnimg force-pushed the dual-stepper-setup-removal branch from 23fc676 to 7a882c8 Compare December 21, 2024 14:14
Copy link

github-actions bot commented Dec 21, 2024

All values in bytes. Δ Delta to base

Target ΔFlash ΔSRAM Used Flash Used SRAM Free Flash Free SRAM
MK3S_MULTILANG 0 0 247756 5654 6196 2538
MK3_MULTILANG 0 0 247126 5663 6826 2529

If either Y_DUAL_STEPPER_DRIVERS or Z_DUAL_STEPPER_DRIVERS is defined
the compilation will fail with an error message. This configuration will likely
never be fully implemented unless someone from the community steps up.
@gudnimg gudnimg force-pushed the dual-stepper-setup-removal branch from 7a882c8 to 2e6d529 Compare December 21, 2024 14:15
@3d-gussner 3d-gussner merged commit 901f63e into prusa3d:MK3 Dec 30, 2024
4 checks passed
@gudnimg gudnimg deleted the dual-stepper-setup-removal branch December 30, 2024 15:16
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.

3 participants