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

env yaml missing tabulate #47

Merged
merged 5 commits into from
Sep 10, 2024
Merged

env yaml missing tabulate #47

merged 5 commits into from
Sep 10, 2024

Conversation

matteoguarrera
Copy link
Contributor

No description provided.

@matteoguarrera
Copy link
Contributor Author

we should use the same requirements between yaml and requirements.txt I noticed also that jax 0.4.20 raise some deprecation error. So I would suggest to push everything to jax 0.4.26. BTW I am also using NERSC

@joglekara
Copy link
Contributor

yeah, agreed... I just use requirements.txt and a venv on NERSC to avoid conda issues on jobs. I think we can just source requirements.txt into the yaml file. Can you implement that? maybe 2 different reqs, one for gpu and one for cpu?

@matteoguarrera
Copy link
Contributor Author

matteoguarrera commented Jul 11, 2024

I actually create a conda environment using yaml on NERSC, so actually there is no need to keep requirements.txt.
You can just conda env create -f env.yml or conda env create -f env_gpu.yml .
Or are you suggesting doing something like this?

name: my_environment
channels:
  - conda-forge
dependencies:
  - python=3.8
  - pip
  - pip:
      -r requirements_cpu.txt  # Replace with requirements_gpu.txt for GPU

@joglekara
Copy link
Contributor

yeah I mean the latter bc I still use reqs.txt :P

But also because it is compatible with both workflows while the conda yaml doesn't support venv.

@joglekara joglekara merged commit c04d4f5 into ergodicio:main Sep 10, 2024
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.

2 participants