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 EIA readme #732

Merged
merged 2 commits into from
Feb 9, 2024
Merged

Update EIA readme #732

merged 2 commits into from
Feb 9, 2024

Conversation

allisonhorst
Copy link
Contributor

No description provided.

@allisonhorst allisonhorst requested a review from Fil February 9, 2024 14:09
Comment on lines +32 to +33
- `eia-bia-reference.csv`: Reference information about balancing authority name, time zone, region, country, etc.
- `eia-connections-reference.csv`: Reference information about connections between balancing authorities
Copy link
Contributor

Choose a reason for hiding this comment

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

it would be nice to have the source URL nonetheless, because when these change in the far future our dashboard will break

Copy link
Contributor

@Fil Fil left a comment

Choose a reason for hiding this comment

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

this should not be a draft PR :)

@allisonhorst allisonhorst marked this pull request as ready for review February 9, 2024 17:27
@allisonhorst allisonhorst enabled auto-merge (squash) February 9, 2024 17:28
@allisonhorst allisonhorst merged commit f52bc22 into main Feb 9, 2024
2 checks passed
@allisonhorst allisonhorst deleted the eia-updates branch February 9, 2024 17:29
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