Skip to content

[CI] Disable PyMVA so we can support the newest TensorFlow versions #19124

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

guitargeek
Copy link
Contributor

Right now we don't support the newest TensorFlow releases after the 2.15 series, initially released in November 2023 [1].

Support for TensorFlow>=2.16 is worked on [2], but it's difficult to make progress if we don't have TensorFlow>=2.16 in our CI images. And we veto it from our CI image via the requirements.txt mechanism.

To continue, we need to temporarily disable PyMVA in the CI, such that we can include the newest TensorFlow version in our images, and then re-enable it in the PR that fixes support for the newest TensorFlow. This is what is done in this commit.

Supporting the newest TensorFlow became a bit more important now that we also have another feature besides PyMVA that uses TensorFlow: the RBatchGenerator. It would not good to hold off testing the actively-developed RBatchGenerator on newer TensorFlow versions, just because PyMVA prevents us from installing these on our CI.

[1] https://github.com/tensorflow/tensorflow/releases
[2] #15790

Right now we don't support the newest TensorFlow releases after the 2.15
series, initially released in November 2023 [1].

Support for TensorFlow>=2.16 is worked on [2], but it's difficult to
make progress if we don't have TensorFlow>=2.16 in our CI images.
And we veto it from our CI image via the `requirements.txt` mechanism.

To continue, we need to temporarily disable PyMVA in the CI, such that
we can include the newest TensorFlow version in our images, and then
re-enable it in the PR that fixes support for the newest TensorFlow.
This is what is done in this commit.

Supporting the newest TensorFlow became a bit more important now that we
also have another feature besides PyMVA that uses TensorFlow: the
RBatchGenerator. It would not good to hold off testing the
actively-developed RBatchGenerator on newer TensorFlow versions, just
because PyMVA prevents us from installing these on our CI.

[1] https://github.com/tensorflow/tensorflow/releases
[2] root-project#15790
@guitargeek guitargeek self-assigned this Jun 22, 2025
@guitargeek guitargeek requested a review from dpiparo as a code owner June 22, 2025 07:56
Copy link

Test Results

    20 files      20 suites   3d 5h 10m 45s ⏱️
 2 999 tests  2 999 ✅ 0 💤 0 ❌
58 414 runs  58 414 ✅ 0 💤 0 ❌

Results for commit 68cae38.

@guitargeek
Copy link
Contributor Author

guitargeek commented Jun 22, 2025

The changes suggested by this PR should only be merged after the tutorial selection system is fixed by this other PR:

Like this, we avoid bad surprises when we eventually enable tmva-pymva again.

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.

1 participant