Skip to content

Conversation

pabloitu
Copy link
Collaborator

@pabloitu pabloitu commented Jul 26, 2023

Addresses #234

pyCSEP Pull Request Checklist

Type of change:

Please delete options that are not relevant.

  • Github Actions Update

Checklist:

  • I have performed a self-review of my own code

@pabloitu pabloitu changed the title ci: changed environment build from miniconda to micromamba ci: changed environment build of pypi-publish from miniconda to micromamba Jul 26, 2023
@pabloitu pabloitu requested review from wsavran and fabiolsilva July 26, 2023 12:45
@pabloitu
Copy link
Collaborator Author

pabloitu commented Jul 26, 2023

I tested this with floatcsep here link

Comment on lines +19 to 25
- uses: mamba-org/setup-micromamba@v1
with:
activate-environment: csep-dev
python-version: '3.10'
channels: conda-forge
generate-run-shell: true
environment-file: requirements.yml
create-args: >-
python=3.10

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good Pablo, this change from conda to mamba speeds-up processing quite a bit. I compared this script against the other CI scripts and it looks good. This should run much faster with micromamba.

@fabiolsilva fabiolsilva merged commit 44da5d3 into SCECcode:master Aug 24, 2023
@pabloitu pabloitu deleted the test_pypi branch August 16, 2024 13:29
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