search

NREL-Sienna / InfrastructureSystems.jl

Utility package for Sienna's simulation infrastructure
https://nrel-sienna.github.io/InfrastructureSystems.jl/
BSD 3-Clause "New" or "Revised" License
38 stars 20 forks source link

Strip module from parameter types during serialization #282

Closed daniel-thom closed 2 years ago

daniel-thom commented 2 years ago

Fixes https://github.com/NREL-SIIP/PowerSystems.jl/issues/863

codecov[bot] commented 2 years ago

Codecov Report

Merging #282 (6e90313) into master (e28f261) will increase coverage by 0.00%. The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master     #282    +/-   ##
========================================
  Coverage   78.27%   78.27%            
========================================
  Files          43       43            
  Lines        3604     3453   -151     
========================================
- Hits         2821     2703   -118     
+ Misses        783      750    -33
Flag Coverage Δ
unittests 78.27% <0.00%> (+<0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/serialization.jl 51.45% <0.00%> (+0.02%) :arrow_up:
src/common.jl 66.66% <0.00%> (-33.34%) :arrow_down:
src/utils/utils.jl 64.58% <0.00%> (-1.57%) :arrow_down:
src/utils/print.jl 52.10% <0.00%> (-1.10%) :arrow_down:
src/utils/generate_structs.jl 84.68% <0.00%> (-0.91%) :arrow_down:
src/internal.jl 78.84% <0.00%> (-0.79%) :arrow_down:
src/probabilistic.jl 77.19% <0.00%> (-0.78%) :arrow_down:
src/scenarios.jl 83.67% <0.00%> (-0.65%) :arrow_down:
src/deterministic.jl 81.53% <0.00%> (-0.56%) :arrow_down:
src/utils/recorder_events.jl 91.39% <0.00%> (-0.53%) :arrow_down:
... and 18 more
kdheepak commented 2 years ago

Thanks for the quick fix. Is it possible to get out a release before noon today? No worries if that's not practical.