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 : The average AQI values on the summary page are inconsistent with those on the city page #286

Open
rstrange-scottlogic opened this issue Aug 5, 2024 · 1 comment

Comments

@rstrange-scottlogic
Copy link
Collaborator

rstrange-scottlogic commented Aug 5, 2024

We have found examples where the AQI shown on the summary page does not match the AQI shown on the city page.

For example Budapest at 31/07/2024 18:00 is showing an Ozone value of 4 and forecast of 2 on the summary page:

Image

But the city page at this time shows an overall AQI of 3 at this time, which is incorrect:

Image

It appears that currently:

  • On the summary we take the average of each pollutant, find the AQI from this, then pick the highest
  • On the city page we find the AQI of each measuring station, and average those

Of these the first is correct.

Whilst investigating I've noticed that the overall AQI difference displayed for Budapest may also be wrong, it looks like it should be showing 4 and 2 as higher AQI numbers but is instead showing 1 and 3, this could be a symptom of the previous bug and will be investigated here.
EDIT : The above is actually working as expected. Where there is a draw on the AQI difference in the first instance we prefer to show the data where the forecast is above the measured.

@rstrange-scottlogic rstrange-scottlogic self-assigned this Aug 5, 2024
@rstrange-scottlogic rstrange-scottlogic changed the title p0 - The average AQI values on the summary page are inconsistent with those on the city page Bug : The average AQI values on the summary page are inconsistent with those on the city page Aug 5, 2024
@mnyamunda-scottlogic
Copy link
Collaborator

mnyamunda-scottlogic commented Aug 7, 2024

Charter executed on Budapest 04/08/2024 00:00 here

@rstrange-scottlogic rstrange-scottlogic removed their assignment Aug 8, 2024
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