Skip to content

Conversation

samuelbles07
Copy link
Collaborator

@samuelbles07 samuelbles07 commented Feb 1, 2025

Problem

On version 3.1.13 and 3.1.21, when monitor finish setup() function, display show 0 values for all measurements for a moment.

Solution

Make sure scheduler for sensor readings run first before display to oled and post to server. By reset display and post data scheduler at the end of setup function.

IMG_9914.1.mp4

@nick-4711
Copy link
Collaborator

Propose to add a description, and maybe also a short video that shows how it behaves now.

@samuelbles07
Copy link
Collaborator Author

samuelbles07 commented Feb 1, 2025

Propose to add a description, and maybe also a short video that shows how it behaves now.

Description update along with the video @nick-4711

@nick-4711
Copy link
Collaborator

I can't see the dashes on the video. The idea was to show a dash ('-') whenever a value is unknown. This can be

  • right after boot
  • when a sensor fails (e.g. if we pull out the S8 then we get a dash for the CO2 value etc.)

@samuelbles07
Copy link
Collaborator Author

samuelbles07 commented Feb 2, 2025

I can't see the dashes on the video. The idea was to show a dash ('-') whenever a value is unknown. This can be

  • right after boot
  • when a sensor fails (e.g. if we pull out the S8 then we get a dash for the CO2 value etc.)

Showing dash symbol when sensor values is invalid already integrated on 3.1.9 (AFAIK). But v3.1.13 problem is, when monitor first show dashboard display on the oled, it shows 0 values instead of dash symbol.

On description video, i am making sure measure reading already happen before showing dashboard display so it will directly show real values. But i think set default to invalid value for each measurement on boot would be better. See this commit.

Result video: https://drive.google.com/file/d/1fVXTNcveLnoolEf77fsaULmuWWnKrBjt/view?usp=drive_link

@nick-4711

@samuelbles07 samuelbles07 merged commit 05dbe60 into develop Feb 3, 2025
20 checks passed
@samuelbles07 samuelbles07 deleted the fix/zero-display branch February 5, 2025 03:01
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

Successfully merging this pull request may close these issues.

2 participants