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.

Sign up for GitHub

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

Jump to bottom

The "today" AirNow site meta file #129

Open
zmoon opened this issue Aug 30, 2023 · 2 comments · May be fixed by #146
Open

The "today" AirNow site meta file #129

zmoon opened this issue Aug 30, 2023 · 2 comments · May be fixed by #146
Labels
bug Something isn't working
Milestone
v0.3

Comments

@zmoon
Copy link
Member

zmoon commented Aug 30, 2023 •
edited
Loading

Currently we use the "today" site meta file (https://s3-us-west-1.amazonaws.com//files.airnowtech.org/airnow/today/monitoring_site_locations.dat) to get site metadata. But earlier periods may involve sites not mentioned in this file.

For example, considering hourly data for the month 2019-08, I found two sites were missing from the today (2023-08-30) file (340130003 and 010735003).
@zmoon zmoon added the bug Something isn't working label Aug 30, 2023
@zmoon
Copy link
Member Author

zmoon commented Sep 6, 2023

Talked to Phil Dickerson about this. Currently, the only way is the conservative way, to read each day's site metadata file (in addition to its data file).

@zmoon
Copy link
Member Author

zmoon commented Sep 14, 2023

Thinking should add an option to this effect, maybe off by default to preserve the current behavior.

@zmoon zmoon added this to the v0.3 milestone Oct 3, 2023
@zmoon zmoon linked a pull request Oct 30, 2023 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
v0.3
Development

Successfully merging a pull request may close this issue.

AirNow metadata options and other updates zmoon/monetio
1 participant
@zmoon