-
Notifications
You must be signed in to change notification settings - Fork 23
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
Add FunctionData
Printing
#377
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #377 +/- ##
==========================================
+ Coverage 77.16% 77.53% +0.36%
==========================================
Files 66 66
Lines 5072 5092 +20
==========================================
+ Hits 3914 3948 +34
+ Misses 1158 1144 -14
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Above formats are unchanged, added a third format that is human-readable but less verbose, for use where |
Re-requesting review for this ^ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Compact looks nice
I implemented some more human-friendly printing of the various
FunctionData
:These could be useful when I do the same thing for the PSY cost structs (NREL-Sienna/PowerSystems.jl#1048).
Also fixed a typo in existing printing test code.