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 AppVeyor config #27960

Merged
merged 1 commit into from Apr 25, 2024
Merged

Update AppVeyor config #27960

merged 1 commit into from Apr 25, 2024

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Mar 21, 2024

PR summary

AppVeyor is quite slow, but is also the only CI to check the conda environment, so cut it down to only one Python version. But also enable a few more things to see if they work.

PR checklist

@QuLogic QuLogic force-pushed the update-appveyor branch 4 times, most recently from 0fb9dbb to ca7e5b7 Compare March 22, 2024 04:22
@QuLogic
Copy link
Member Author

QuLogic commented Mar 22, 2024

I tried enabling miktex and imagemagick, and though conda-forge has both of them, the latter doesn't seem to be available. And miktex fails with what seems to be a permission error. I tried to investigate over RDP, but couldn't find anything obvious other than maybe the package itself is missing some config files. So, we can try to enable that again later, and get the cut down setup in now.

@ksunden
Copy link
Member

ksunden commented Mar 22, 2024

Hmmm... Exactly one SVG test is failing with a very small RMS (0.004)

@QuLogic
Copy link
Member Author

QuLogic commented Mar 23, 2024

Looking at the results, it's a small bit of text:
image
but this is not visible if you compare the .svg files directly. And it also seems to be 1 bit or so, as it's not particularly visible in the failed-diff.png output.

So I'll just add some fuzz on this one.

@dstansby dstansby added the CI: testing CI configuration and testing label Mar 24, 2024
AppVeyor is quite slow, but is also the only CI to check the conda
environment, so cut it down to only one Python version. But also enable
a few more things to see if they work.
@QuLogic QuLogic added this to the v3.9.0 milestone Apr 25, 2024
@ksunden ksunden merged commit 5f40ecf into matplotlib:main Apr 25, 2024
42 of 44 checks passed
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Apr 25, 2024
@QuLogic QuLogic deleted the update-appveyor branch April 26, 2024 02:44
QuLogic added a commit that referenced this pull request Apr 26, 2024
…960-on-v3.9.x

Backport PR #27960 on branch v3.9.x (Update AppVeyor config)
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

3 participants