-
-
Notifications
You must be signed in to change notification settings - Fork 110
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
Add Q4 CEMS data #3379
Add Q4 CEMS data #3379
Conversation
Do we want to update the |
I think we need to match the same year of CEMS and 860 so the plants are all matched, so I was deferring to comparing CEMS to the full year of EPA data rather than just the 860M data? |
Ok that's probably a good idea. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good - I wanted to try running it locally to be sure, but it's taking a long time, so I'm going to approve! If you've loaded it locally / checked the tables to make sure everything is there as expected, I think you can go ahead and add it to the queue.
Overview
Closes #3315
What problem does this address?
Adds Q4 2023 CEMS data.
What did you change?
Added Q4 data to extraction, updated docs.
Testing
How did you make sure this worked? How can a reviewer verify this?
Materialize the EPA CEMS dagster assets.
To-do list