Closed MathewBiddle closed 2 years ago
Thanks for the report, Mathew. I think we're dealing with a few issues here:
setup.md
._includes/scripts
isn't the right place for this script either (it's hard to find). If we want participants to use this script, it should be provided in the ZIP file that we provide on the setup page.check_env.py
, we can discuss removing the "Anaconda" check.I like your suggestion to add environment.yml -- please feel free to submit a PR if you're still up to it.
The ggplot/plotnine transition happened a while back (in #248). It might be an indication that no one has actually used that script since 2018... 🤔
Addressed in #493.
In preparing for my teaching demo next week :flushed: I've been reviewing the instructor notes for this lesson and came across two problems with check_env.py.
python check_env.py
it fails with the messageAssertionError: Anaconda is NOT installed as your default version of Python. Please make sure that is in accordance with the instructions provided.
Anaconda
(note the capital A). As that is now how the structure is created with the Anaconda distribution. There is probably a cleaner way to do this, however I think that will fix for now.ggplot
required for this workshop? Shouldn't that beplotnine
? https://github.com/datacarpentry/python-ecology-lesson/blob/7a2319fd3d15f04608e999010904b2a7a485a50c/_includes/scripts/check_env.py#L27I can submit a PR to make these changes, if they are appropriate.
As a side note, it might be helpful to provide an environment.yml file in the setup as well. I looked around and couldn't find anything (although there is a lot of material to cover here, so I might have missed it). If it's of interest, this is what I have so far:
Which could be built with
$ conda env create -f .\environment.yml