Skip to content

Conversation

timurbazhirov
Copy link
Member

No description provided.

cd ${{ matrix.name }}
# Source the variables from the CF build script: https://github.com/conda-forge/pytorch-cpu-feedstock/blob/main/recipe/build.sh
source ../scripts/build-pytorch.sh
Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe use as another parameter - {{ matrix.script }}

Nit - build-pytorch.sh to `pytorch-build.sh if we end up adding other packages in the future

- name: Build with Pyodide
run: |
set -vxeuo pipefail
set -vxeo pipefail
Copy link
Member Author

Choose a reason for hiding this comment

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

We should probably fix the version of Python also somewhere to be the same as in Pyodide (3.11.2 at this time, I believe)

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