The Wayback Machine - https://web.archive.org/web/20240106173914/https://github.com/matplotlib/matplotlib/pull/27539
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

Update conda on AppVeyor #27539

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

oscargus
Copy link
Contributor

PR summary

The AppVeyor tests are pretty slow and quite a bit of the time seems to be setting things up. (About 12 of 36 minutes spent running the actual tests.)

As a really old version of conda is used by default (4.7.12), this PR aims to upgrade to the latest conda version which hopefully runs faster (as most things are faster).

PR checklist

@oscargus oscargus added the CI: testing CI configuration and testing label Dec 18, 2023
@oscargus
Copy link
Contributor Author

oscargus commented Dec 18, 2023

Seems like the problem is that the image that we are using only has Python 3.7 installed so resolving the environment for 3.9 and 3.10 takes significant time. https://www.appveyor.com/docs/windows-images-software/#python Also, updating conda does not give the latest version, so not clear how much of the recent improvements are actually included.

Updating the image will only give 3.8 though if we stick with conda. pip-based installs can run any version of Python. But then, I do not really know what the purpose of the AppVeyor tests are.

@tacaswell
Copy link
Member

Could micromamba help us here?

It looks like the upgrade is only getting us from 4.12 -> 4.14 which is still well before the change to use the mamba solver. My guess (not checked) is that 4.14 is the newest version available for py37.

@@ -55,6 +55,7 @@ install:
- conda config --set always_yes true
- conda config --set show_channel_urls yes
- conda config --prepend channels conda-forge
- conda update -n base -q conda
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- conda update -n base -q conda
- conda update -n base -q conda python

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This should only update python to the newest 3.7 afaIk.

@QuLogic
Copy link
Member

QuLogic commented Dec 19, 2023

It looks like we could use Chocolatey to get a newer miniconda3 with Python 3.9, but the latest version is still 4.12.0. Chocolatey also has mambaforge 23.3.1.100, which should be about the same as miniforge.

@timhoffm
Copy link
Member

If Appveyor does not provide a recent minconda out of the box, micromamba is likely the way to go.

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

Successfully merging this pull request may close these issues.

None yet

4 participants