search

carpentries-incubator / deep-learning-intro

Learn Deep Learning with Python
https://carpentries-incubator.github.io/deep-learning-intro/
Other
31 stars 36 forks source link

Replace anaconda #513

Closed carschno closed 3 months ago

carschno commented 4 months ago

Merge queue: should be merged after #512

Closes #511, using Pip/Venv installation.

github-actions[bot] commented 4 months ago

Thank you!

Thank you for your pull request :smiley:

:robot: This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

Rendered Changes

:mag: Inspect the changes: https://github.com/carpentries-incubator/deep-learning-intro/compare/md-outputs..md-outputs-PR-513

The following changes were observed in the rendered markdown documents:

 md5sum.txt |   2 +-
 setup.md   | 175 ++++++++++++++++++++++++-------------------------------------
 2 files changed, 70 insertions(+), 107 deletions(-)
What does this mean? If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible. This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

:stopwatch: Updated at 2024-08-06 10:34:15 +0000

svenvanderburg commented 3 months ago

@tobyhodges, @psteinb, or @colinsauze are you on Windows and could one of you test these new installation instructions?

colinsauze commented 3 months ago

@tobyhodges, @psteinb, or @colinsauze are you on Windows and could one of you test these new installation instructions?

I'm on Linux, I do have Windows installed but not booted it in a while. I'll try and boot it later today and see if I can test things out.

svenvanderburg commented 3 months ago

No need to test it again on Windows as @DaniBodor already did it! I think we can move forward with this.

svenvanderburg commented 3 months ago

@carschno do you think we could merge this?