Open frapac opened 5 years ago
Good Point! Although, I must admit I'm not having much luck with the building of the weave reports in the documentation when I remove those visualization packages. Does anyone have a suggestion on how to organize the project tomls of the package? @Thuener @odow @joaquimg
You can add the solvers just for the tests, e.g., https://github.com/odow/SDDP.jl/blob/0490bea2c46787e1d4d63a5491ea0106c7fe70cf/Project.toml#L24-L30
I haven't tried optional dependencies.
You can add the solvers just for the tests, e.g., https://github.com/odow/SDDP.jl/blob/0490bea2c46787e1d4d63a5491ea0106c7fe70cf/Project.toml#L24-L30
I haven't tried optional dependencies.
Even if I run them also for the documentation?
Could the vizualization packages simply be testing dependencies first? Later on we try to set them as optional.
Fixed test packages to extra in 3916f4afcefb6f367a08b79303f096dde2390aae . Although, it seams that, since the visualization functions are inside the module, I'm not able to move them to extra. The only other package outside the scope of the module is Weave, but some how the target to "docs" folder is not being successful : https://travis-ci.com/andrewrosemberg/HydroPowerModels.jl/builds/119367814
Looking at the file Project.toml, it occurs to me that we might want to reduce the dependencies of HydroPowerModels.jl.
For instance, do we need
Ipopt.jl
,Weave.jl
,Cairo.jl
orGR.jl
inProject.toml
? By looking at the code in the module, it seems that we only needPowerModels.jl
,JuMP.jl
,GLPK.jl
andSDDP.jl
as main dependencies. Maybe we could move the other packages in[extras]
for testing purposes?Furthermore, do you think we could add
Plots.jl
as an optional dependency (for instance by exporting the plotting functions only ifPlots.jl
is already installed)? When deporting some computations in a cluster, we might want to load only the minimal set of packages IMO.X-ref https://github.com/JuliaCon/proceedings-review/issues/35