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

refactor: add mwmiles calculations to design.transmission module #291

Merged
merged 2 commits into from
Sep 10, 2020

Conversation

danielolsen
Copy link
Contributor

@danielolsen danielolsen commented Sep 10, 2020

Purpose

Moves analyses of scenario design (i.e., analyses that only work on the Grid or ChangeTable and do not require any results) from PostREISE to PowerSimData. See https://github.com/Breakthrough-Energy/RenewableEnergyProject/issues/307 and Breakthrough-Energy/PostREISE#162.

What is the code doing

There is no new code, only code from PostREISE being added (into the new powersimdata/design/transmission folder), renamed (powersimdata.design.transmission -> powersimdata.design.transmission.upgrade), or import corrections.

Validation

All tests pass.

Time to review

10 minutes.

@rouille
Copy link
Collaborator

rouille commented Sep 10, 2020

Do you plan to create the generation folder in a different PR?

@danielolsen
Copy link
Contributor Author

Do you plan to create the generation folder in a different PR?

Is it worth creating the folder now without putting anything in it? I was thinking I would wait until some generation design analysis code is ready for a PR.

@rouille
Copy link
Collaborator

rouille commented Sep 10, 2020

Do you plan to create the generation folder in a different PR?

Is it worth creating the folder now without putting anything in it? I was thinking I would wait until some generation design analysis code is ready for a PR.

I thought there was already modules in powersimdata.design related to generation. We will create the folder later then.

@danielolsen
Copy link
Contributor Author

I thought there was already modules in powersimdata.design related to generation. We will create the folder later then.

You are right, clean_capacity_scaling.py is definitely designing generation, as is mimic_grid.py (although in the future I would also like to mimic transmission as well, so maybe it is okay to keep here). Maybe I should create that folder now after all.

@rouille
Copy link
Collaborator

rouille commented Sep 10, 2020

I thought there was already modules in powersimdata.design related to generation. We will create the folder later then.

You are right, clean_capacity_scaling.py is definitely designing generation, as is mimic_grid.py (although in the future I would also like to mimic transmission as well, so maybe it is okay to keep here). Maybe I should create that folder now after all.

If I understand, powersimdata.design/clean_capacity_scaling can go in a new powersimdata/design/generation folder.

Can we create a powersimdata/design/mimic folder where mimic_grid can go and later mimic_transmission?

@danielolsen
Copy link
Contributor Author

I thought there was already modules in powersimdata.design related to generation. We will create the folder later then.

You are right, clean_capacity_scaling.py is definitely designing generation, as is mimic_grid.py (although in the future I would also like to mimic transmission as well, so maybe it is okay to keep here). Maybe I should create that folder now after all.

Done.

@danielolsen
Copy link
Contributor Author

Can we create a powersimdata/design/mimic folder where mimic_grid can go and later mimic_transmission?

In my head, I'm thinking that a mimic/mimic_grid module would have functions for both mimic_generation(), mimic_transmission(), possibly also mimic_demand() etc., with one mimic_all() function or similar that would do everything. We could break them into separate files if necessary, but maybe it'd all be short enough to stay in one file and avoid another folder level.

Copy link
Collaborator

@rouille rouille left a comment

Choose a reason for hiding this comment

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

Looks good

Copy link
Collaborator

@BainanXia BainanXia left a comment

Choose a reason for hiding this comment

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

Reading through the discussion, I like the plan. Thanks for taking care of this.

@danielolsen danielolsen force-pushed the daniel/design_analyses branch from d008ef4 to 34f0257 Compare September 10, 2020 21:35
@danielolsen danielolsen force-pushed the daniel/design_analyses branch from 34f0257 to 518f32c Compare September 10, 2020 21:36
@danielolsen danielolsen merged commit 684dd81 into develop Sep 10, 2020
@danielolsen danielolsen deleted the daniel/design_analyses branch September 10, 2020 22:40
@ahurli ahurli mentioned this pull request Mar 11, 2021
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.

4 participants