NREL-Sienna / PowerSystems.jl

Data structures in Julia to enable power systems analysis. Part of the Scalable Integrated Infrastructure Planning Initiative at the National Renewable Energy Lab.
https://www.nrel.gov/analysis/sienna.html
BSD 3-Clause "New" or "Revised" License
299 stars 71 forks source link

Add Pretty Cost Printing #1125

Closed GabrielKS closed 3 weeks ago

GabrielKS commented 3 weeks ago

Fixes https://github.com/NREL-Sienna/PowerSystems.jl/issues/1048. Depends on https://github.com/NREL-Sienna/InfrastructureSystems.jl/pull/377.

Here's how the new cost struct printouts look at their most complicated (printing some operational costs):

Screenshot 2024-06-12 at 5 11 50 PM

Next is to figure out a way of abbreviating this to stick it into the components printout.

GabrielKS commented 3 weeks ago

Okay, now components show more cost information than before but still a very abbreviated summary to more or less fit on one line:

Screenshot 2024-06-14 at 1 09 32 PM Screenshot 2024-06-14 at 1 10 01 PM Screenshot 2024-06-14 at 1 10 19 PM
GabrielKS commented 3 weeks ago

Is "(a type of InputOutputCurve)" necessary? We haven't focused on InputOutputCurve elsewhere in documentation, so I think it could be extraneous and confusing

My thinking was we want to make it super clear when things are input-output curves versus incremental/marginal curves, etc., but I agree that using type names that aren't otherwise very exposed is confusing. Thoughts on another way to do this, or do you think it's already sufficiently clear that this is an input-output curve?