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

FNIRSI GC-01: Wrong device temperature statistics #28

Closed
Alter0ne opened this issue Mar 16, 2024 · 20 comments
Closed

FNIRSI GC-01: Wrong device temperature statistics #28

Alter0ne opened this issue Mar 16, 2024 · 20 comments
Labels
bug Something isn't working

Comments

@Alter0ne
Copy link

Hello!
I have the device with ch32f103 processor. It shows the incorrect temperature: 9 °C when the environment is 28 °C.
Photos
Also my device resets the current date and time after rebooting. But the time zone is saved correctly. Perhaps it's my battery problem?

@Gissio
Copy link
Owner

Gissio commented Mar 16, 2024

Regarding the real-time clock. Date and time should keep running after poweroff. Check the button battery's voltage (it's a CR1220): it should be around 3 V. If it doesn't, replace it.

And the temperature discrepancy is expected. STM32F103 and compatible microprocessors are not temperature calibrated in the factory. This means there can be a variation of up to ± 20 °C from the actual value, because the temperature sensor of different production batches can be physically different.

To fix this, there should be a setting to calibrate this. That's actually a question I have been asking mytself. And this question goes to everyone who reads this. Does it make sense to show uncalibrated temperature? Should there be settings to fix this? In order to keep things simple, I think it would be best to not show temperature at all (if it's wrong, it makes no sense as it confuses more than helping).

@martin-machacek
Copy link

My device shows the temperature 13 to 15 degrees Celsius higher (ambient temp is 24, GC shows 38). In my opinion it doesnt make sense to show it, if there is no way to calibrate.

@Gissio
Copy link
Owner

Gissio commented Mar 16, 2024

Agreed. Next beta will replace device temperature with device battery voltage (which makes more sense).

@Gissio Gissio closed this as completed Mar 16, 2024
@Gissio
Copy link
Owner

Gissio commented Mar 16, 2024

I just released beta21 which fixes this. Please let me know if it works.

https://github.com/Gissio/radpro/releases

@martin-machacek
Copy link

Unfortunately it turns on and the screen is blank (probably the background color, does not seem to be white). I can hear particle clicks and particle LED flashes, but nothing else. Key presses don't help.

@Gissio
Copy link
Owner

Gissio commented Mar 16, 2024

My bad. Try beta22 now!

@martin-machacek
Copy link

beta22 works and shows the battery voltage.
4.539V connected to USB, after disconnecting it slowly settles for cca 10 minutes down to 3.780 and than slower as it discharges.

@Gissio
Copy link
Owner

Gissio commented Mar 16, 2024

Nice!

@Gissio
Copy link
Owner

Gissio commented Mar 16, 2024

Can I make a request? It would be great if you can measure the discharge curve of the battery so the battery indicator shows correct values.

This would imply fully charging the device, and taking note of the voltage value every 10 minutes, until the voltage drops to 2.7 V. It would be great if you do this with the screen off, turning it on only to read the voltage. Can you do this?

@martin-machacek
Copy link

I can try tomorrow. I am thinking of ways to automate this. Wouldn't it be possible to log it internally?

@Gissio
Copy link
Owner

Gissio commented Mar 16, 2024

It would be, but I think it takes less time to do this manually than for me to program the code.

@martin-machacek
Copy link

Ok, I will give it a try tomorrow then

@Tig3rch3n
Copy link

I can test both of my devices with the power profiler in a few days.
I even plan to do different Setups

Brightness max/Display on all the Time vs Brighness min/Sleep after 30sec and a few in between

@Gissio
Copy link
Owner

Gissio commented Mar 16, 2024

A measurement on one device should be enough.

Use only brightness max/display on all time so the measurement takes less time.

Thanks in advance!

@Gissio Gissio added the bug Something isn't working label Mar 16, 2024
@Gissio Gissio changed the title FNIRSI GC-01: wrong device temperature statistics FNIRSI GC-01: Wrong device temperature statistics Mar 16, 2024
@Gissio Gissio reopened this Mar 16, 2024
@Tig3rch3n
Copy link

Btw... would it be possible to enable a Temp Offset?
I have a big'ish diff in both directions

Roomtemp 23°
Device 1 14°
Device 2 27.5°

@martin-machacek
Copy link

Temp offset setting would be nice.

I am still collecting the voltage values. The battery lasted whole day yesterday and still has 3.533V. After having it switched off for the night I have charged it up for 30s and when the voltage dropped back to the last value from yesterday, I have resumed logging.

@Tig3rch3n
Copy link

Tig3rch3n commented Mar 18, 2024

Same, i did a Sleep after 30s run yesterday and do a always on min brightness at the moment :)
I'll get the Power Profiler in a few days, so i'll do that maybe over the Easter Holidays

@martin-machacek
Copy link

Here is my voltage log: Fnirsi-GC-01-voltage-log.txt
The device turned off after the last value of 3.042V
Parameters during test:

  • Interval 10 minutes
  • Display sleep after 30 seconds
  • Display brightness level High
  • Rad Pro version 2.0beta22

@Gissio
Copy link
Owner

Gissio commented Mar 18, 2024

They say an image is worth a thousand words.

fnirsi-gc01-battery-discharge

Thanks for all the hard work!

@Gissio
Copy link
Owner

Gissio commented Mar 22, 2024

Updated curve is available in 2.0rc01!

@Gissio Gissio closed this as completed Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants