Closed GabrielKS closed 2 months ago
@daniel-thom thoughts on automatic @scoped_enum
serialization support along these lines?
Attention: Patch coverage is 96.29630%
with 5 lines
in your changes are missing coverage. Please review.
Project coverage is 75.52%. Comparing base (
e87f7b8
) to head (4d72144
).
@daniel-thom thoughts on automatic
@scoped_enum
serialization support along these lines?
NIce. We can remove some similar code from PSY.
Is this PR still in draft or is it ready for review?
I think it's ready for review, assuming we don't end up needing to make any more changes here related to the market bid cost.
@GabrielKS please merge as soon as a tracking issue with the remaining TODO's is open please. Same for https://github.com/NREL-Sienna/PowerSystems.jl/pull/1071.
Tracking issue is https://github.com/NREL-Sienna/InfrastructureSystems.jl/issues/352, I'll merge this as soon as I've verified tests pass.
Draft for now, this is the companion to https://github.com/NREL-Sienna/PowerSystems.jl/pull/1071.