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 EPA/EIA crosswalk of plant 55641 #389

Merged
merged 1 commit into from
Aug 21, 2024
Merged

Conversation

grgmiller
Copy link
Collaborator

@grgmiller grgmiller commented Aug 9, 2024

Purpose

Generators CTG3, CTG4, STG2, and PV1 were proposed to be added to the Riverside Energy Center (Plant 55641) and were in testing as of 2019 with scheduled COD in 2020. However, starting in the 2020 EIA data, these generators disappeared from this plant, and reappeared as the West Riverside Energy Center (Plant 64020). Closes CAR-4543.

2019 Proposed Generator Data:
image

2020 Operational Generator Data:
image

I've updated the manual EPA-EIA crosswalk file to represent this. STG2 does not appear to directly show up in the data

Testing

We would expect the subplant crosswalk to reflect the new mapping.

As expected, the EPA 55641 is no longer associated with those two units at EIA 55641 (although they still appear in the crosswalk, which is strange since they were never built)
image
These two units are now associated with plant 64020, and are correctly linked to the same subplant as STG2
image

Review estimate

5 min

Future work

What issues were identified that are not being addressed in this PR but should be addressed in future work?

Checklist

  • Update the documentation to reflect changes made in this PR
  • Format all updated python files using black
  • Clear outputs from all notebooks modified
  • Add docstrings and type hints to any new functions created

@grgmiller grgmiller changed the base branch from main to greg/update_s3_zipper August 9, 2024 22:18
@grgmiller grgmiller marked this pull request as ready for review August 9, 2024 22:20
@grgmiller grgmiller requested a review from gailin-p August 9, 2024 22:20
@grgmiller
Copy link
Collaborator Author

@rouille you're supposed to be on vacation!

Copy link
Collaborator

@gailin-p gailin-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I assume all the other line changes are line ending? Might be nice to switch to unix line endings here too

Base automatically changed from greg/update_s3_zipper to development August 21, 2024 16:09
@grgmiller grgmiller merged commit 8d4dc3e into development Aug 21, 2024
2 checks passed
@grgmiller grgmiller deleted the greg/plant_55641 branch August 21, 2024 16:10
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.

3 participants