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

MPC method follow-up #2807

Closed

Conversation

kisslorand
Copy link
Contributor

@kisslorand kisslorand commented Jun 19, 2023

Requirements

  • MKS or BTT TFT
  • Marlin with MPC enabled

Description

PR #2791 implemented the choice of MPC algorithm method. Selecting the preferred MPC method makes the whole screen flicker. This PR fixes it.

There's also a minuscule, tiny, itsy-bitsy, 8 byte FW size reduction due to removing an if conditional by relocating a "busy processing" splash screen function call in the MPC module.

Benefits

No flickering screen selecting the preferred MPC method.

Related Issues

None reported, own finding.

@stale
Copy link

stale bot commented Sep 16, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Abandoned label Sep 16, 2023
@kisslorand
Copy link
Contributor Author

kisslorand commented Sep 16, 2023

Bump!

@stale stale bot removed the Abandoned label Sep 16, 2023
Copy link

stale bot commented Dec 15, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added Abandoned and removed Abandoned labels Dec 15, 2023
Copy link

This PR has been automatically marked as stale because it has had no activity for the last 60 days. It will be closed in 7 days if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the Stale label Apr 21, 2024
@github-actions github-actions bot closed this Apr 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant