GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Fix some CI errors by bnavigator · Pull Request #244 · python-control/Slycot

Conversation

@bnavigator

@bnavigator bnavigator commented Dec 23, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Fixes:

  • Ubuntu dropped the libopenblas-base "transition" package
  • Mambaforge is deprecated
  • Windows mamba commands silently fail
  • macOS conda MKL error during pytest collection

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
bnavigator added 5 commits December 23, 2024 11:56
@bnavigator
New Ubuntu: libopenblas package changed
9c1039f
@bnavigator
Remove deprecated mambaforge
a7c0232
@bnavigator
Update bash command
85b6153
@bnavigator
replace mamba with conda
d0b4fad
Once again they broke something: conda-incubator/setup-miniconda#371
@bnavigator
macos runners are now arm64. There is no Intel MKL on Arm
af3fe2d
@bnavigator bnavigator mentioned this pull request Dec 23, 2024
CI: Remove MKL form macOS tests, Fix Ubuntu openBLAS install python-control/python-control#1083
Merged
@bnavigator
bnavigator marked this pull request as ready for review December 23, 2024 14:15
@murrayrm
murrayrm merged commit a2ee35a into python-control:master Dec 23, 2024
@slivingston

slivingston commented Dec 24, 2024

Copy link
Copy Markdown
Member

Windows mamba commands silently fail

@bnavigator I looked in the recent CI logs, but I did not find evidence about the mamba command failing silently. Can you show me?

Also, I am motivated to ask because a mamba command is used similarly in python-control CI jobs: https://github.com/python-control/python-control/blob/a2c3b4c8db5558181357d2762e2285ae2ea01dc5/.github/workflows/os-blas-test-matrix.yml#L312

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@roryyorke

roryyorke commented Dec 24, 2024

Copy link
Copy Markdown
Collaborator

looks good, thanks @bnavigator .

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bnavigator

bnavigator commented Dec 24, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator Author

Yes, I noticed that it works on the python-control jobs. But it failed here: https://github.com/python-control/Slycot/actions/runs/12467342357/job/34797033932#step:7:27 The commands just printed out the current windows version (!?)

See conda-incubator/setup-miniconda#371

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bnavigator

bnavigator commented Dec 24, 2024

Copy link
Copy Markdown
Collaborator Author

Oh, it also does not work on the python-control jobs: https://github.com/python-control/python-control/actions/runs/12468173712/job/34799709028#step:7:27
The installs for slycot do not happen. Notice how the tests are automatically skipped by pytest: https://github.com/python-control/python-control/actions/runs/12468173712/job/34799709028#step:8:165

slivingston reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bnavigator bnavigator mentioned this pull request Dec 24, 2024
Replace mamba with conda again python-control/python-control#1085
Merged
@roryyorke roryyorke mentioned this pull request Sep 7, 2025
Release 0.6.1 #248
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@bnavigator @slivingston @roryyorke @murrayrm
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.