Replace Requires.jl with Pkg extensions for collect_results
?
#417
Labels
running-listing
Functionality for running and listing simulation runs
collect_results
?
#417
At the moment the
collect_results
functionality is done using Requires.jl to conditionally load code on using DataFrames.jl.Julia 1.10 and later versions have the Package extensions system which is more robust and natively supported. Doing the switch from Requires.jl to Pkg is honestly trivially easy, here is an example of doing it for e.g., TimeseriesSurrogates.jl for a plotting conditional dependency:
https://github.com/JuliaDynamics/TimeseriesSurrogates.jl/blob/main/src/plotting/surrogate_plot.jl and https://github.com/JuliaDynamics/TimeseriesSurrogates.jl/blob/main/ext/TimeseriesSurrogatesVisualizations.jl and https://github.com/JuliaDynamics/TimeseriesSurrogates.jl/blob/main/Project.toml#L27
Should we do the switch? It would require later versions of DrWatson to raise the Julia dependency from 1.0 to 1.10. I don't know what the impact of this is for the average DrWatson user, as I've always used latest stable version... How many honestly use 1.0 only, and more importantly, why?
The text was updated successfully, but these errors were encountered: