-
-
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
Update gridpathratoolkit
and vceregen
metadata
#3897
Conversation
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.
Not blocking but maybe we should use Elaine's public email since she's the individual being credited?
src/pudl/metadata/constants.py
Outdated
@@ -194,6 +194,14 @@ | |||
"zenodo_role": "producer", | |||
"organization": "Vibrant Clean Energy", | |||
}, | |||
"elaine-hart": { | |||
"title": "Elaine Hart", | |||
"email": "info@momentenergyinsights.com", |
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.
Any reason not to use Elaine's public email? It's on the about page elaine@momentenergyinsights.com
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.
No reason other than that I didn't see it, I'll update!
…m/catalyst-cooperative/pudl into update-gridpathratoolkit-metadata
Overview
What problem does this address?
path
forvceregen
to point towards the website, as the documentation is getting archived. Reversing the decision made in Add source metadata forvceregen
#3887 to use the PDF link.contributors
forgridpathratoolkit
as the first step of Update metadata and licensing for gridpathratoolkit dataset #3895, in order to enable archiving usingpudl-archiver
and a GCS bucket.What did you change?
sources.py
pudl-archiver
repository, but it's also the reason that I'm not currently listing Catalyst as a contributor to the gridpathratoolkit data - we would show up co-listed with Elaine without the ability to specify that we are "distributing" the data rather than creating it.