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

chore: merge develop into master for v0.4.3 release #514

Merged
merged 31 commits into from
Jun 30, 2021
Merged

Conversation

danielolsen
Copy link
Contributor

@danielolsen danielolsen commented Jun 30, 2021

Pull Request doc

Purpose

With #507 and #512, we moved functionality from PostREISE into PowerSimData. With Breakthrough-Energy/PostREISE#310, we remove this functionality from PostREISE and import these functions directly from PowerSimData. If we create a v0.4.3 release of PowerSimData that includes these functions, then we can specify that the upcoming v0.5.0 release of PostREISE requires PowerSimData >= v0.4.3, so that anyone using the latest PostREISE will know that they need a compatible version of PowerSimData.

What the code is doing

All code has been previously reviewed.

Proposed release notes

Closed Issues

Merged pull requests (features)

Merged pull requests (fixes, etc.)

Time estimate

5 minutes.

danielolsen and others added 29 commits June 15, 2021 12:33
…pacity

feat: add substation transmission capacity calculation
…_passthrough

feat: allow state method passthrough in MockScenario objects
…zone_capacity

Josephine/calculate interzone capacity
…st_base_grid

feat: add base_grid input to investment cost calculation functions
refactor: add compare module, move Grid/Scenario check functions from PostREISE to PowerSimData
feat: deduplicate interconnect passed to grid constructor
…pers

refactor: move PostREISE helper functions to PowerSimData
@danielolsen danielolsen requested a review from rouille June 30, 2021 17:40
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.

Thanks

@codecov-commenter
Copy link

codecov-commenter commented Jun 30, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@122516d). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #514   +/-   ##
=========================================
  Coverage          ?   74.08%           
=========================================
  Files             ?       98           
  Lines             ?     7602           
  Branches          ?      983           
=========================================
  Hits              ?     5632           
  Misses            ?     1790           
  Partials          ?      180           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 122516d...e480a62. Read the comment docs.

@danielolsen danielolsen merged commit f8c2347 into master Jun 30, 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.

6 participants