Skip to content

Conversation

@xylar
Copy link
Collaborator

@xylar xylar commented Nov 14, 2024

No description provided.

@xylar
Copy link
Collaborator Author

xylar commented Nov 14, 2024

@altheaden, could you have a quick look here and make sure I didn't miss anything?

@xylar
Copy link
Collaborator Author

xylar commented Nov 14, 2024

I'm also adding python 3.13 support on conda-forge here:
conda-forge/mpas_tools-feedstock#114

channel-priority: strict
auto-update-conda: true
python-version: ${{ matrix.python-version }}
python-version: 3.12
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Apparently conda-build isn't available for python 3.13 yet. That's fine, we don't need to build with the full python matrix, we just need the package that gets built to use the full python matrix.

matrix:
python-version: ["3.9", "3.10", "3.11", "3.12"]
python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
fail-fast: false
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@altheaden, I've been seeing CI get cancelled when one job in the matrix fails. I think this (rather than cancel_others: false below) might be the fix.

Could you add this on other repos when you do other updates?

Comment on lines -71 to +73
conda create -y -n mpas_tools_dev --use-local mpas_tools \
python=${{ matrix.python-version }} \
conda create -y -n mpas_tools_dev -c ${CONDA_PREFIX}/conda-bld/ \
-c conda-forge mpas_tools python=${{ matrix.python-version }} \
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm not sure why python 3.13 likes this better but I'll take it...

@altheaden
Copy link
Collaborator

Looks good to me! If you would like me to do any testing myself, let me know. Otherwise, I think it looks fine to merge.

@xylar xylar merged commit 2653338 into MPAS-Dev:master Nov 14, 2024
5 checks passed
@xylar xylar deleted the add-python-3.13 branch November 14, 2024 23:19
@xylar
Copy link
Collaborator Author

xylar commented Nov 14, 2024

Thanks, @altheaden

@xylar xylar added ci Changes that affect github actions CI and removed continuous integration labels Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes that affect github actions CI conda package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants