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

[BUG] Stat tile on page Today using custom data binding does not appear if the same observation does not exist in main data binding #258

Closed
bidord opened this issue Mar 2, 2024 · 3 comments · Fixed by #259
Labels
bug Something isn't working

Comments

@bidord
Copy link
Contributor

bidord commented Mar 2, 2024

Describe the bug

A stat tile on page Today targetting an observation for a custom data binding does not appear if there is no data for the same observation on the main data binding.

To Reproduce

weewx.conf:

[StdArchive]
  data_binding = wx_binding
  # ...

[DataBindings]
  [[wx_binding]]
    # ...
  [[wx2_binding]]
    # ...
  • wx_binding has data for inTemp but no data for extraTemp1
  • wx2_binding has data for both inTemp and extraTemp1

skin.conf:

[ObservationBindings]
  [[wx2_inTemp]]
    data_binding = wx2_binding
    observation = inTemp
  [[wx2_extraTemp1]]
    data_binding = wx2_binding
    observation = extraTemp1

[DisplayOptions]
  stat_tile_observations = inTemp, wx2_inTemp, wx2_extraTemp1

Page "Today" has a stat tile for inTemp and wx2_inTemp, but the stat tile for wx2_extraTemp1 does not appear.

On other pages the three stat tiles appear as expected.

Expected behavior

The stat tile for wx2_extraTemp1 should appear on page "Today".

@bidord bidord added the bug Something isn't working label Mar 2, 2024
bidord added a commit to bidord/weewx-wdc that referenced this issue Mar 2, 2024
@Daveiano
Copy link
Owner

Daveiano commented Mar 3, 2024

Thank you for the detailed report and the PR, it is very appreciated!

The change looks reasonable to me, did you test this by yourself? Do all observations show up as expected with the fix?

@bidord
Copy link
Contributor Author

bidord commented Mar 4, 2024

You are welcome. I'm glad I can make a contribution to this amazing project!

Yes, I've been testing this on my "production" weewx instance for the last 2 days.

I have a Vantage Pro 2 station. I recently bought an Ecowitt GW2000 to be used as a secondary station in order to add a bunch of extra temperature/humidity sensors. That's how I noticed this bug on the "Today" page.

I can confirm that with this patch:

  • The original bug does not occur anymore ( the extraTempX and extraHumidX of my secondary station do show up on every page where they should).
  • So far, everything else seems to keep working as expected.

@Daveiano
Copy link
Owner

Daveiano commented Mar 4, 2024

Merged.

Thanks again for the detailed report and the PR!

Daveiano added a commit that referenced this issue Mar 4, 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

Successfully merging a pull request may close this issue.

2 participants