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

Update text and documentation to reflect change in the calculation of minimum/maximum Tmin and Tmax over longer periods #77

Open
RutgerDankers opened this issue Dec 7, 2021 · 2 comments

Comments

@RutgerDankers
Copy link

Hi Charlotte

Issue HM6 Daily Maximum Temperature - Monthly Mean and Daily Minimum Temperature - Monthly Mean:

The seasonal and annual aggregation should be based on daily rather than monthly temperature statistics. For example, the selection in the screenshot below would provide the change in the “annual maximum of daily minimum temperature” (i.e. the warmest night of the year) rather than the “annual mean of the monthly maximum of daily minimum temperature” (which is difficult to understand for most users).

Changes have been made in the workflows for Tmax and Tmin (so far only the detailed view app) to address this issue. These changes have not been published yet.

The order of the calculation has been changed, for example in case of the statistic 'maximum of daily maximum temperature', this is what is now being calculated:

image

So if the selected statistic is 'maximum', the data plotted will the be maximum Tmax (or Tmin) over the aggregation period selected, i.e. a month, a season, or the entire year, which I think addresses HM's issue.

The dynamically generated titles of the timeseries plots have been changed to reflect the new statistic, e.g.

image

I still need to check whether this change also needs to be pulled through to the overview apps.

@charliepascoe
Copy link
Collaborator

Fixed with this update to the json file 3330283

@charliepascoe
Copy link
Collaborator

charliepascoe commented Dec 16, 2021

@RutgerDankers I suggest we close this issue when the update is implemented on the ECDE site

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