Skip to content

Fix the cache key for v5 release and master environment#156

Merged
BoxiLi merged 1 commit into
qutip:mainfrom
BoxiLi:fix-publish-version
May 12, 2026
Merged

Fix the cache key for v5 release and master environment#156
BoxiLi merged 1 commit into
qutip:mainfrom
BoxiLi:fix-publish-version

Conversation

@BoxiLi
Copy link
Copy Markdown
Member

@BoxiLi BoxiLi commented Apr 29, 2026

Description
The conda environment cache key used qutip-${{ matrix.qutip-version }}, which evaluated to the same string qutip-5 for both the v5-release and v5-master jobs. Because the conda cache stores the entire environment directory, including pip-installed packages, whichever job saved its cache last would set the qutip version for both jobs on subsequent runs. As a result, qutip-v5-release does not get its own environment but uses the qutip-v5-master env instead.

Changed the cache key suffix from qutip-${{ matrix.qutip-version }} to ${{ matrix.label }}, giving each job (v4-release, v5-release, v5-master) its own independent conda environment cache.

Copy link
Copy Markdown
Contributor

@hodgestar hodgestar left a comment

Choose a reason for hiding this comment

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

Oops. This probably explains a bunch of strange flakiness in the test runs. :/

@BoxiLi
Copy link
Copy Markdown
Member Author

BoxiLi commented Apr 29, 2026

There is something that I still don't understand.

According to the qutip.about(), some notebooks are using the release version, some the dev version

https://nbviewer.org/urls/qutip.org/qutip-tutorials/tutorials-v5/quantum-circuits/qip-toffoli-cnot.ipynb
At this moment uses qutip 5.3.0.dev0+c9f293f while

https://nbviewer.org/urls/qutip.org/qutip-tutorials/tutorials-v5/quantum-circuits/teleportation.ipynb
uses qutip 5.2.3

@BoxiLi BoxiLi marked this pull request as ready for review April 29, 2026 13:46
@BoxiLi BoxiLi merged commit 1edb1e8 into qutip:main May 12, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants