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

Creality Ender 3 Pro / CR-10 S5 / 427 + BLTouch #1026

Merged
merged 9 commits into from
May 1, 2024

Conversation

JPT77
Copy link
Contributor

@JPT77 JPT77 commented Feb 29, 2024

Requirements

Creality printers with STM32 version 4.2.7 board and BL touch sensor.

Description

This config adds a BL touch config to the above two creality printers . It is not 100% final but it's already quite stable.
I added a lot of comments with "JPT" which explain why I set some variables this way and where more work my be needed. I also added a read-me and fetched the logo files from existing configs.
The files are meant for current 2.1-bugfix branch, not the 2.1.2.2 release. Should I deliver configs for the latest release too? Where?

Benefits

Will be easier to set up BL touch for these printers.

Disclaimer

It's 10 years since I last fiddled with Marlin. So please check thoroughly before accepting this PR.

@thinkyhead
Copy link
Member

It is best that the configuration with BLTouch match as closely as possible the parent config in all aspects that are not probe-related. If there are improvements to be made to a given machine configuration, it should be made to all of the machine's configurations.

@thinkyhead thinkyhead changed the title add config for Creality Ender 3 Pro and CR10-S5 both with 32 bit 427 board and BLtouch sensor Creality Ender 3 Pro / CR-10 S5 / 427 + BLTouch Mar 14, 2024
@thinkyhead
Copy link
Member

Is the bed temp sensor actually different on this model (1 instead of 5), with a higher max temp?

@thinkyhead
Copy link
Member

thinkyhead commented Mar 14, 2024

  • When using UBL the mesh should occupy the entire bed area (minus a safe margin).
  • As you noted, the MAX POS values should almost always be larger than the BED SIZE values. Those should be corrected in the standard configs also. I just did this for X MAX only so far.
  • Although your machine can't handle a high BAUD RATE we should leave it alone and stick with the defaults. The same goes for any other adjustments that are not universally required.

@JPT77
Copy link
Contributor Author

JPT77 commented Mar 15, 2024

Is the bed temp sensor actually different on this model (1 instead of 5), with a higher max temp?

I don't know.
I tried checking the bed temp with an infrared thermometer and it seemed to be almost 10 °C off. Seemed to be less with the new config. but not sure.

  • When using UBL the mesh should occupy the entire bed area (minus a safe margin).

minus save margin or minus "reachable"?
ie, you you mean it should include positions not reachable by the probe that should be taken manually?

  • As you noted, the MAX POS values should almost always be larger than the BED SIZE values. Those should be corrected in the standard configs also. I just did this for X MAX only so far.

not sure what that is about, already forgot ;)

  • Although your machine can't handle a high BAUD RATE we should leave it alone and stick with the defaults. The same goes for any other adjustments that are not universally required.

I agree. seems to be another issue on my part.

did you change everything you mentioned here or should I?

@thinkyhead
Copy link
Member

minus save margin or minus "reachable"?

The "safe margin" is a margin that ensures that the probe gets good readings when it probes the bed. Inductive probes don't do well near the edge, and some cheaper beds have more outlier heights nearer to the edge.

Which points are reachable will depend on where the probe is at the extremes. If the probe can't reach some edge of the bed, we don't care for UBL. With UBL the points the probe can't reach can be probed afterward with the nozzle paper test. For this reason the UBL mesh should cover almost the entire bed, minus that "safe margin" to ensure the probe doesn't freak out at the edges.

@thinkyhead thinkyhead merged commit c771a54 into MarlinFirmware:import-2.1.x May 1, 2024
1 check passed
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.

2 participants