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

docs: Note "NTC 100k beta 3950" as not preferred #4789

Closed

Conversation

crashmaxx
Copy link

docs: Note "NTC 100k beta 3950" as not preferred

As pointed out in issue #4054, the "NTC 100k beta 3950" sensor
type is not as accurate as "EPCOS 100K B57560G104F" for most
NTC 100k B3950 thermistors. The generic name and inclusion
next to other sensor types with more specific names, implies
that it would be a good default option.

To guide users away from this option, this PR removes it from
the [extruder] section, and lists it at the end of the options in
the "Common thermistors" section with "not preferred" in
parenthesis next to it.

Signed-off-by: Andrew Stowell [email protected]

docs: Note "NTC 100k beta 3950" as not preferred

As pointed out in issue Klipper3d#4054, the "NTC 100k beta 3950" sensor
type is not as accurate as "EPCOS 100K B57560G104F" for most
NTC 100k B3950 thermistors. The generic name and inclusion
next to other sensor types with more specific names, implies
that it would be a good default option.

To guide users away from this option, this PR removes it from
the [extruder] section, and lists it at the end of the options in
the "Common thermistors" section with "not preferred" in
parenthesis next to it.

Signed-off-by: Andrew Stowell <[email protected]>
@grigi
Copy link
Contributor

grigi commented Oct 11, 2021

One minor comment, the docs doesn't make it clear, that if your thermistor says its a "beta 3950k" (like many cheap ones do) that you should probably use the EPCOS spec.

@KevinOConnor
Copy link
Collaborator

Thanks. I think we may just want to deprecate the existing "NTC 100k beta 3950" - see my comments at #4054 (comment)

-Kevin

@KevinOConnor
Copy link
Collaborator

I think this should be addressed with the merge of PR #4859 .

-Kevin

@KevinOConnor KevinOConnor added the resolved Issue is thought to now be fixed label Nov 19, 2021
@crashmaxx crashmaxx closed this Nov 19, 2021
@crashmaxx crashmaxx deleted the NTC-100k-beta-3950-not-preferred branch November 19, 2021 15:31
@github-actions github-actions bot locked and limited conversation to collaborators Nov 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
resolved Issue is thought to now be fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants