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

Temperature offset only in sensor memory #178

Closed
Olaruci opened this issue Feb 15, 2024 · 3 comments
Closed

Temperature offset only in sensor memory #178

Olaruci opened this issue Feb 15, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@Olaruci
Copy link

Olaruci commented Feb 15, 2024

CO2 Gadget version and flavour

Paste the two first serial log lines, as:

-->[STUP] CO2 Gadget Version: 0.9.030-development beta: TTGO T-Display 
-->[STUP] Version compiled: Feb 09 2024 

CO2 Gadget installation method?

  • Easy Web flasher?

Describe the bug

If the offset is stored in the sensor (Sensirion SDC30 ) but CO2 Gadget doesn't know it, it can be out of sync and the sensor still adjust the temperature but CO2 Gadget shows the offset at 0.

To Reproduce
Steps to reproduce the behavior:
it can happen if, for example, you adjust the offset in CO2 Gadget (and so in the sensor) but later you reset CO2 Gadget preferences by, for example, erasing the flash memory when installing new version.
Then every time after a restart I can see a temperature of 30C that slowly decreases to an ambiental temp of 21, so clearly an offset is applied but in Co2 Gadget is showed as 0 (where previously it had to be 9 to have same behavior)

Expected behavior
Temperature offset inside the sensor and in Co2 Gadget to be in sync.

@melkati melkati self-assigned this Feb 15, 2024
@melkati melkati added the bug Something isn't working label Feb 15, 2024
@melkati
Copy link
Owner

melkati commented Feb 18, 2024

Thank you reporting and for your patience,

Now the offset is working right with some fixes we applied internally to the under the hood sensors library kike-canaries/canairio_sensorlib#198 but there is still some issue with temperature offset to be on sync between [sensor <=> library <=> CO2 Gadget].

I'm still working on it. Version 0.9.36 on development already includes some changes but it's not finished.

Sorry for the time it's taking, it's not too easy as it's deeply hidden in the logic and looks like it not only affects the SCD30 sensor but also the SCD4x.

@melkati
Copy link
Owner

melkati commented Feb 19, 2024

Hi.

Just to let you know I'm still working on this.

Looks like the bug is deeper than I thought. You can find more information and see advancements here: kike-canaries/canairio_sensorlib#203

Thank you very much for uncovering it.

@melkati
Copy link
Owner

melkati commented Jun 7, 2024

I believe this issue has been resolved. Closing now.

@melkati melkati closed this as completed Jun 7, 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
Development

No branches or pull requests

2 participants