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

Updated ender 5 configuration #1087

Closed
wants to merge 4 commits into from

Conversation

PeaceNira
Copy link
Contributor

Requirements

  • Filling out this template is required. Pull Requests without a clear description may be closed at the maintainers' discretion.

Description

the config has been updated to 2.1.2.4 with minor format changes, also fixed the high current that is not needed for the stock ender 5 motors.

Benefits

fixed motor current values

Related Issues

not a bug, more of a optimization to reduce unneeded wear/heat.

@thisiskeithb
Copy link
Member

There's nothing that needs to be "brought up to date" since the import-2.1.x and bugfix-2.1.x branches are already the most up to date branches. By overwriting those with 2.1.2.4 configs, you're breaking support for everyone/the next version.

Also, motor currents are already correct since stock motors are rated from the manufacturer using peak values instead of RMS:

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.

2 participants