codecov[bot]
commented
4 years ago Codecov Report
Merging #13 into master will decrease coverage by
0.40%. The diff coverage is54.90%.
@@ Coverage Diff @@
## master #13 +/- ##
==========================================
- Coverage 35.80% 35.40% -0.41%
==========================================
Files 11 11
Lines 458 627 +169
==========================================
+ Hits 164 222 +58
- Misses 294 405 +111 | Impacted Files | Coverage Δ | |
|---|---|---|
| src/PowerModelsAnalytics.jl | 100.00% <ø> (ø) |
|
| src/layouts/networkx.jl | 100.00% <ø> (ø) |
|
| src/plots/analytics.jl | 0.00% <0.00%> (ø) |
|
| src/plots/graph.jl | 0.00% <0.00%> (ø) |
|
| src/plots/networks.jl | 0.00% <0.00%> (ø) |
|
| src/layouts/common.jl | 40.00% <33.33%> (ø) |
|
| src/core/data.jl | 73.33% <74.57%> (+2.74%) |
:arrow_up: |
| src/graph/common.jl | 76.43% <76.43%> (-17.97%) |
:arrow_down: |
| src/core/types.jl | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update b0bc719...f132252. Read the comment docs.
This includes a substantial rewrite of PowerModelsAnalytics to make it more agnostic to the type of Infrastructure network that is attempting to be plotted. A lot was attempted to be abstracted away in order to allow the user to provide custom specifications for things like nodes, edges, sources, demands, etc., without having to write new functions to plot those types of Infrastructure networks. That being said, this is only currently tested with PowerModels and PowerModelsDistribution.
This PR specifically adds support for the PowerModelsDistribution ENGINEERING data model, as well as maintaining support for the MATHEMATICAL model (which is used by both PowerModels and PowerModelsDistribution).
Many function kwargs were changed to make them more generic to the type of network, and default values were moved in many cases to
src/core/options.jlso it wasn't any longer necessary to monitor many functions across several files to keep them in sync.The load-block variants of the plot and build graph function was removed in favor of using the kwarg
block_graph=true, mainly because the amount of code replication was huge and it was easier to condense down into a single function once the functions were refactored to make them more agnostic to the type of network.An equivalent of plot_load_summary called plot_source_demand_summary specifically for the ENGINEERING model was added. It has some abstractions in it already and may be able to be adapted for other types of Infrastructure networks later, but focuses right now on power.