search

ANL-CEEESA / UnitCommitment.jl

Optimization package for the Security-Constrained Unit Commitment Problem
Other
104 stars 24 forks source link

Fix missing Distributions dependency #15

Closed akazachk closed 3 years ago

akazachk commented 3 years ago

To fix #14

iSoron commented 3 years ago

Hi @akazachk, thank you for the PR, but note that Distributions is already listed in Project.toml, and that this PR is just reordering the lines in this file. The file that really got modified when you ran resolve and instantiate was Manifest.toml, but we no longer include it in the repository, as it makes the package incompatible with different versions of Julia.

akazachk commented 3 years ago

Got it. So to get make to work, you have to run instantiate in both the main and benchmark directories first.

iSoron commented 3 years ago

That's right. Not sure why can't Julia do this automatically. I think removing the stale Manifest.toml file might also work.