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

[BUG] CR-10 Example Profile Thermistor Incorrect #15039

Closed
Deneteus opened this issue Aug 24, 2019 · 11 comments
Closed

[BUG] CR-10 Example Profile Thermistor Incorrect #15039

Deneteus opened this issue Aug 24, 2019 · 11 comments

Comments

@Deneteus
Copy link

Deneteus commented Aug 24, 2019

Description:

The thermistor type for the CR-10 and CR-10S is incorrect in the Example profile.

The thermistor for the heat bed is listed as 100K ohm NTC 3950FB. In the original CR-10 source Creality used 1 for both the bed and the hot end. These are also the only thermistors they sell for this printer.

Steps to Reproduce

Please review the Example Configuration files for the CR-10 and CR-10S and review the thermistor settings for the bed and hot end. You will noticed the bed thermistor is set to 5.

https://github.com/MarlinFirmware/Marlin/blob/1.1.x/Marlin/example_configurations/Creality/CR-10/Configuration.h

image

https://github.com/MarlinFirmware/Marlin/blob/1.1.x/Marlin/example_configurations/Creality/CR-10S/Configuration.h

image

Additional Information

Creality CR-10 Github Config.h
https://github.com/Creality3DPrinting/CR-10/blob/master/Firmware/Marlin/Configuration.h

Official Site:
https://www.creality3donline.com/creality-5pcs-100k-ohm-ntc-3950fb-thermistors-with-cable-temperature-sensor-reprap-extruder-for-creality-3d-printer-mend-hot-bed_p0035.html

image

image

@boelle boelle changed the title [FR] CR-10 Example Profile Thermistor Incorrect [BUG] CR-10 Example Profile Thermistor Incorrect Aug 24, 2019
@Deneteus Deneteus changed the title [BUG] CR-10 Example Profile Thermistor Incorrect [FR] CR-10(S) Example Profile Bed Thermistor Incorrect Aug 24, 2019
@Deneteus Deneteus changed the title [FR] CR-10(S) Example Profile Bed Thermistor Incorrect [BUG] CR-10(S) Example Profile Bed Thermistor Incorrect Aug 24, 2019
@Deneteus Deneteus changed the title [BUG] CR-10(S) Example Profile Bed Thermistor Incorrect [BUG] CR-10 Example Profile Thermistor Incorrect Aug 24, 2019
@Deneteus
Copy link
Author

Thanks Boelle. :)

@boelle
Copy link
Contributor

boelle commented Aug 24, 2019

no problemo :-)

@thisiskeithb
Copy link
Member

See PR #15040.

thisiskeithb added a commit to thisiskeithb/Marlin that referenced this issue Aug 24, 2019
The CR-10 & CR-10S use 100K ohm NTC 3950 thermistors. Fixes MarlinFirmware#15039.
@boelle
Copy link
Contributor

boelle commented Aug 24, 2019

so we can "swing the axe" on this one when the PR gets merged :-D

@thisiskeithb
Copy link
Member

so we can "swing the axe" on this one when the PR gets merged :-D

Yep.

These changes will be implemented in bugfix-2.0.x since 1.1.x is no longer receiving support.

@Deneteus
Copy link
Author

Bugfix-2.0.x is what I was testing when I noticed that the thermistor value was off.

@thisiskeithb
Copy link
Member

Bugfix-2.0.x is what I was testing when I noticed that the thermistor value was off.

Ahh. You linked 1.1.x, so I was just making sure 😄

@sl1pkn07
Copy link
Contributor

then if affect the creality printers, the ender series is affected?

@thisiskeithb
Copy link
Member

thisiskeithb commented Aug 24, 2019

Looking at the Creality firmware for those machines, they should be updated as well. I'll update my PR.

edit: Now updated.

@github-actions
Copy link

github-actions bot commented Jul 4, 2020

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Jul 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants