Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

updating to sim-store branch #19

Merged
merged 14 commits into from
Feb 2, 2021
Merged

updating to sim-store branch #19

merged 14 commits into from
Feb 2, 2021

Conversation

claytonpbarrows
Copy link
Member

@claytonpbarrows claytonpbarrows commented Jan 8, 2021

No description provided.

@claytonpbarrows
Copy link
Member Author

There is currently a bug somewhere that causes the following issue:

julia> include("script/3_PowerSimulations_examples/01_operations_problems.jl")
.
.
.
[ Info: Unit System changed to SYSTEM_BASE
ERROR: LoadError: MethodError: no method matching root_module(::Nothing)
Closest candidates are:
  root_module(::Module, ::Symbol) at loading.jl:969
  root_module(::Base.PkgId) at loading.jl:968
Stacktrace:
 [1] root_module(::Module, ::Symbol) at ./loading.jl:969
 [2] get_type_from_strings(::String, ::String) at /Users/cbarrows/.julia/packages/InfrastructureSystems/ZLiq3/src/utils/utils.jl:348
 [3] _read_time_series_type(::HDF5.Group) at /Users/cbarrows/.julia/packages/InfrastructureSystems/ZLiq3/src/hdf5_time_series_storage.jl:207
 [4] (::InfrastructureSystems.var"#104#105"{Deterministic,InfrastructureSystems.Hdf5TimeSeriesStorage,InfrastructureSystems.DeterministicMetadata,UnitRange{Int64},UnitRange{Int64}})(::HDF5.File) at /Users/cbarrows/.julia/packages/InfrastructureSystems/ZLiq3/src/hdf5_time_series_storage.jl:327
 [5] h5open(::InfrastructureSystems.var"#104#105"{Deterministic,InfrastructureSystems.Hdf5TimeSeriesStorage,InfrastructureSystems.DeterministicMetadata,UnitRange{Int64},UnitRange{Int64}}, ::String, ::Vararg{String,N} where N; swmr::Bool, pv::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}) at /Users/cbarrows/.julia/packages/HDF5/d0V7K/src/HDF5.jl:507
 [6] h5open at /Users/cbarrows/.julia/packages/HDF5/d0V7K/src/HDF5.jl:505 [inlined]
 [7] deserialize_time_series(::Type{Deterministic}, ::InfrastructureSystems.Hdf5TimeSeriesStorage, ::InfrastructureSystems.DeterministicMetadata, ::UnitRange{Int64}, ::UnitRange{Int64}) at /Users/cbarrows/.julia/packages/InfrastructureSystems/ZLiq3/src/hdf5_time_series_storage.jl:323
 [8] get_time_series(::Type{Deterministic}, ::VariableReserve{ReserveDown}, ::String; start_time::DateTime, len::Nothing, count::Int64) at /Users/cbarrows/.julia/packages/InfrastructureSystems/ZLiq3/src/component.jl:164
 [9] get_time_series(::PowerSimulations.PSIContainer, ::VariableReserve{ReserveDown}, ::String) at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/devices_models/devices/common/get_time_series.jl:11
 [10] service_requirement_constraint!(::PowerSimulations.PSIContainer, ::VariableReserve{ReserveDown}, ::ServiceModel{VariableReserve{ReserveDown},RangeReserve}) at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/services_models/reserves.jl:36
 [11] construct_service!(::PowerSimulations.PSIContainer, ::Array{VariableReserve{ReserveDown},1}, ::System, ::ServiceModel{VariableReserve{ReserveDown},RangeReserve}, ::Dict{Symbol,DeviceModel}, ::Array{DataType,1}) at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/services_models/services_constructor.jl:97
 [12] (::PowerSimulations.var"#_construct_valid_services!#357"{PowerSimulations.PSIContainer,System,Dict{Symbol,DeviceModel},Array{DataType,1}})(::ServiceModel{VariableReserve{ReserveDown},RangeReserve}) at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/services_models/services_constructor.jl:33
 [13] construct_services!(::PowerSimulations.PSIContainer, ::System, ::Dict{Symbol,ServiceModel}, ::Dict{Symbol,DeviceModel}) at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/services_models/services_constructor.jl:51
 [14] _build!(::PowerSimulations.PSIContainer, ::OperationsProblemTemplate, ::System) at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/core/operations_problem.jl:428
 [15] build! at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/core/operations_problem.jl:404 [inlined]
 [16] OperationsProblem at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/core/operations_problem.jl:84 [inlined]
 [17] OperationsProblem{GenericOpProblem}(::OperationsProblemTemplate, ::System, ::Nothing; kwargs::Base.Iterators.Pairs{Symbol,Any,Tuple{Symbol,Symbol},NamedTuple{(:optimizer, :horizon),Tuple{MathOptInterface.OptimizerWithAttributes,Int64}}}) at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/core/operations_problem.jl:70
 [18] #OperationsProblem#43 at /Users/cbarrows/.julia/packages/PowerSimulations/vglZh/src/core/operations_problem.jl:58 [inlined]
 [19] top-level scope at /Users/cbarrows/Documents/repos/SIIPExamples.jl/script/3_PowerSimulations_examples/01_operations_problems.jl:117
 [20] include(::String) at ./client.jl:457
 [21] top-level scope at REPL[2]:1
in expression starting at /Users/cbarrows/Documents/repos/SIIPExamples.jl/script/3_PowerSimulations_examples/01_operations_problems.jl:117

@claytonpbarrows claytonpbarrows marked this pull request as ready for review February 2, 2021 04:54
@claytonpbarrows
Copy link
Member Author

@rodrigomha and @jd-lara can you review the changes to the PSID examples in this commit
It looks like there are conflicts between PSI and PSID or some other reason that I needed to add PSID. to several function calls.

@codecov
Copy link

codecov bot commented Feb 2, 2021

Codecov Report

Merging #19 (1d1ebbb) into master (3c69c71) will decrease coverage by 0.28%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   22.78%   22.50%   -0.29%     
==========================================
  Files           1        1              
  Lines          79       80       +1     
==========================================
  Hits           18       18              
- Misses         61       62       +1     
Impacted Files Coverage Δ
src/SIIPExamples.jl 22.50% <0.00%> (-0.29%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4392959...1d1ebbb. Read the comment docs.

@claytonpbarrows claytonpbarrows merged commit 5d7ca49 into master Feb 2, 2021
@claytonpbarrows claytonpbarrows deleted the cb/sim_store branch February 2, 2021 15:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant