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

Fix buggy environment yml #14

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

trevorcampbell
Copy link
Collaborator

@trevorcampbell trevorcampbell commented Sep 13, 2023

This is a WIP to try to deal with OS-dependent environment setup

Unfortunately it appears there is no way to make environment.yml files OS-dependent, and it's not easy to install packages that aren't on conda forge (e.g. tidyclust). So I think we need to move to a more generic solution for environment setup when using JupyterLab Desktop.

This PR:

  • removes the github action that dumps the conda environment from the docker image (won't work on windows due to unixodbc, doesn't capture packages not from conda)
  • makes the code from the 1st edition conda env update --file ... spit out an error message with instructions for what to do instead
  • [todo] adds new install code elsewhere

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New environment.yml generation is buggy unixodbc install bug on windows 10
1 participant