vprusso / toqito

|toqito> (Theory of Quantum Information Toolkit) in Python :snake:
https://toqito.readthedocs.io/en/latest/
MIT License
141 stars 56 forks source link

Why is `macos-latest` job for Python3.10 leading to an error? #576

Closed purva-thakre closed 14 hours ago

purva-thakre commented 5 months ago

Noticed the following when all the jobs passed successfully in a PR. The jobs for macos in Python3.10 give an error that we should upgrade to the latest version of pip.

image

Can't really figure out why the error specifies python3.10 -m pip install --upgrade pip when both jobs use python -m pip install --upgrade pip

https://github.com/vprusso/toqito/blob/e0aed37964889f706201d12e5a3f2ca07e69e5b4/.github/workflows/build-test-actions.yml#L39

https://github.com/vprusso/toqito/blob/e0aed37964889f706201d12e5a3f2ca07e69e5b4/.github/workflows/build-test-actions.yml#L73

purva-thakre commented 5 months ago

Interestingly, Mitiq also gets similar warnings for macos.

image https://github.com/unitaryfund/mitiq/actions/runs/8899188557

purva-thakre commented 5 months ago

@vprusso Should we downgrade the macos-latest version in the runner? I think the notice is related to https://github.com/actions/runner-images/issues/9770#issue-2271788870

purva-thakre commented 5 months ago

Per https://github.com/unitaryfund/mitiq/issues/2339#issuecomment-2088294257, this issue might also need to be closed. Will update later today or tomorrow.

purva-thakre commented 5 months ago

Went over this with Vincent. The Notice does not make sense. If we click on one of the Annotations in Mitiq, it provides a more verbose output for how the runner is installing things.

The runner manages to upgrade pip on its own, also states the latest version of pip was installed but later follows it up with the notice in the screenshot.

https://github.com/unitaryfund/mitiq/actions/runs/8899188557/job/24437954297#step:3:28

image image image

vprusso commented 15 hours ago

Are we able to close this one out, @purva-thakre ?

Don’t think this one is still an issue but maybe I’m wrong?