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

Thermistor Temperatures are way off. #7

Closed
CuriousTimo opened this issue Feb 3, 2022 · 2 comments
Closed

Thermistor Temperatures are way off. #7

CuriousTimo opened this issue Feb 3, 2022 · 2 comments

Comments

@CuriousTimo
Copy link

CuriousTimo commented Feb 3, 2022

I have replaced an SKR mini v1.1 with the TinyBee.

#define TEMP_SENSOR_0 1
#define TEMP_SENSOR_BED 1

The temperature on the TinyBee is way off. It reads about 73 degrees for both the bed and E0 whereas the SKR board reads about 18 degrees.
This is at ambient temperature with the printer just sitting idle so I can confirm that the temperature is about 18 degrees.

When I disconnect the thermistors the temperatures read 70 degrees.

Gcode M149 confirms that the printer is set to Celcius.
The thermistors are standard 100K and the printer is using a 12V power supply

@CuriousTimo
Copy link
Author

Found the issue.

In src\HAL\ESP32\HAL.cpp line 237 has been commented out in the MKS repository.
The value has been changed from 3300.0 to 2600.0

I am using the standard Marlin 2.9.0.3 repo. So after doing the same change my thermistors are showing the correct temps.

233 void HAL_adc_start_conversion(const uint8_t adc_pin) {
234 const adc1_channel_t chan = get_channel(adc_pin);
235 uint32_t mv;
236 esp_adc_cal_get_voltage((adc_channel_t)chan, &characteristics[attenuations[chan]], &mv);
237 // HAL_adc_result = mv * 1023.0 / 3300.0;
238 HAL_adc_result = mv * 1023.0 / 2600.0;

@infoheleno
Copy link

muito grato: CuriosoTimo
consegui resover o mesmo problema pela sua ajuda!

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

No branches or pull requests

2 participants