Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

For reproducibility, include a Manifest.toml file with all examples. #137

Closed
ablaom opened this issue May 21, 2019 · 0 comments · Fixed by #142
Closed

For reproducibility, include a Manifest.toml file with all examples. #137

ablaom opened this issue May 21, 2019 · 0 comments · Fixed by #142
Labels
good first issue Good for newcomers

Comments

@ablaom
Copy link
Member

ablaom commented May 21, 2019

It is unrealistic to expect examples, as they proliferate, will be kept up-to-date with the packages they use (including MLJ).

Please, let us give each script (notebook) in examples/ its own directory, and include a Manifest.toml we know to be compatible. When an example is updated, so is the Manifest.

At present we will also need to include the Project.toml; its seems that you cannot directly instantiate a lone Manifest.toml (JuliaLang/Pkg.jl#1091).

I believe the correct code to add to the header of each script, to ensure the accompanying Manifest.tom is instantiated, is:

using Pkg
Pkg.activate(@__DIR__)
Pkg.instantiate()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant