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

Added example config for Ender5Pro 422 with Micro-Swiss Direct Drive and BLtouch #759

Merged

Conversation

JMLdev
Copy link
Contributor

@JMLdev JMLdev commented Jun 21, 2022

There is currently no example configuration for Ender 5 Pro with BL touch. It took quite some time to figure out all of these settings and I would like to save someone else the effort. There are a few tweaks in my config files, but they're explained in the README along with instructions to change them back to stock. These are the values that I arrived at after running through two days of intense testing. This build has been tested and confirmed with my Ender 5 Pro with BLtouch and Micro-Swiss direct drive.

@JMLdev
Copy link
Contributor Author

JMLdev commented Jun 21, 2022

This also has long filename support enabled, and a setting to disable software endstops for calibration of the BL touch probe.

@JMLdev
Copy link
Contributor Author

JMLdev commented Jun 21, 2022

This has a few advanced menus added to the default interface for convenience.

I removed double probing because it's annoying and takes too long.

@JMLdev
Copy link
Contributor Author

JMLdev commented Jun 22, 2022

@thisiskeithb I addressed your recommendation. Please resolve conversation. Thanks.

@thisiskeithb
Copy link
Member

I addressed your recommendation. Please resolve conversation. Thanks.

Only thinkyhead can resolve conversations and he'll take care of it when he does a final pass before merging.

@JMLdev
Copy link
Contributor Author

JMLdev commented Jun 22, 2022

I addressed your recommendation. Please resolve conversation. Thanks.

Only thinkyhead can resolve conversations and he'll take care of it when he does a final pass before merging.

Okay cool. Thanks for the review.

@JMLdev
Copy link
Contributor Author

JMLdev commented Jun 25, 2022

Lowered jerk to a more reasonable amount based on further testing and added HOST_ACTION_COMMANDS for OktoPrint support.

@thinkyhead thinkyhead force-pushed the ender5-422-bltouch-msdd branch from bf065ee to 94f2ab5 Compare June 29, 2022 19:59
@thinkyhead thinkyhead force-pushed the ender5-422-bltouch-msdd branch from 94f2ab5 to 82c2179 Compare June 29, 2022 21:28
@thinkyhead
Copy link
Member

Instead of making a separate configuration for all this particular combination of add-on hardware, I've opted here to just add a couple of conditionals and apply the other enhancements to the existing CrealityV422 config. I reverted the E-steps to 93 because a hotend alone should not be changing that number.

@JMLdev
Copy link
Contributor Author

JMLdev commented Jul 1, 2022

Instead of making a separate configuration for all this particular combination of add-on hardware, I've opted here to just add a couple of conditionals and apply the other enhancements to the existing CrealityV422 config. I reverted the E-steps to 93 because a hotend alone should not be changing that number.

It isn't just a hotend, it's the direct drive conversion, which changes the gear ratio, and therefore the esteps.

@JMLdev
Copy link
Contributor Author

JMLdev commented Jul 17, 2022

Is anyone going to complete this pull request or request revisions?

@thinkyhead thinkyhead merged commit 23c3bb0 into MarlinFirmware:import-2.1.x Jul 28, 2022
@thisiskeithb
Copy link
Member

It isn't just a hotend, it's the direct drive conversion, which changes the gear ratio, and therefore the esteps.

Since this has been merged, you'll need to submit a new PR with this change. Just wrap it in #if ENABLED(ENDER5_USE_MICROSWISS) and use an #else for the old values.

@thisiskeithb
Copy link
Member

It isn't just a hotend, it's the direct drive conversion, which changes the gear ratio, and therefore the esteps.

I've submitted #790 to take care of this.

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