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

Normals #79

Merged
merged 61 commits into from
Sep 24, 2019
Merged

Normals #79

merged 61 commits into from
Sep 24, 2019

Conversation

steffilazerte
Copy link
Member

Big changes

  • New function: normals_dl() function downloads climate normals. Addresses issue Add normals data #38.
  • New argument: stations_search() has normals_only to return only stations with climate normals
  • Deprecated url argument in favour of weathercan.urls.stations, weathercan.urls.weather and weathercan.urls.normals options.
  • Deprecated tz_disp in favour of time_disp. Now all timezones are UTC, but the displayed time is either local time or UTC. When time_disp = "none", the time displayed is local time without daylight savings, similar to how ECCC presents the data. This means that data from different time zones will have similar ecological times (i.e. midnights will be comparable), but the actual times are not UTC. When time_disp = "UTC', the time displayed is in UTC timezone, meaning that stations from different times zones will have true times (i.e. midnight observation in Toronto will be three hours before midnight observation in Vancouver). Addresses issue Switch to UTC #74.

Small changes

Bug fixes

@steffilazerte steffilazerte merged commit 7eb004c into master Sep 24, 2019
@steffilazerte steffilazerte deleted the normals branch September 24, 2019 20:35
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