-
Notifications
You must be signed in to change notification settings - Fork 705
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
{lib}[foss/2022a] TensorFlow v2.13.0 #18644
{lib}[foss/2022a] TensorFlow v2.13.0 #18644
Conversation
…orFlow-2.13.0_add-default-shell-env.patch, TensorFlow-2.13.0_add-missing-snappy-function.patch, TensorFlow-2.13.0_add-missing-system-absl-py-target.patch, TensorFlow-2.13.0_add-missing-system-protobuf-targets.patch, TensorFlow-2.13.0_exclude-xnnpack-on-ppc.patch, TensorFlow-2.13.0_fix-protobuf-compatibility.patch, TensorFlow-2.13.0_fix-pybind11_protobuf.patch, TensorFlow-2.13.0_remove-io-gcs-filesystem-dep.patch, TensorFlow-2.13.0_remove-libclang-dep.patch, TensorFlow-2.13.0_revert-to-flatbuffers-2.0.6.patch, TensorFlow-2.13.0_unpin-gast-version.patch
@Flamefire: Tests failed in GitHub Actions, see https://github.com/easybuilders/easybuild-easyconfigs/actions/runs/5976856309
bleep, bloop, I'm just a bot (boegelbot v20200716.01) |
Test report by @Flamefire |
Test report by @Flamefire |
TF 2.13 requires I could double-check if that still works anyway or we could allow a 2nd protobuf version as we e.g. also have 2 versions for GCCcore-10.2: https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/p/protobuf/protobuf-2.5.0-GCCcore-10.2.0.eb https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/p/protobuf/protobuf-3.14.0-GCCcore-10.2.0.eb and the switch from 3.19 to "v21" is a similar major change Edit: I tested with the existing protobuf-python 3.19 and |
Test report by @casparvl |
Looks like an environment issue to me, e.g. not enough space, job killed, ... |
('nsync', '1.25.0'), | ||
('SQLite', '3.38.3'), | ||
('patchelf', '0.15.0'), | ||
('protobuf-python', '4.21.9'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This introduces a new variant of protobuf-python
as dependency in the 2022a
generation of easyconfigs.
We can add an exception in the test suite for this if needed, but is it worth pursuing this since we already have TensorFlow 2.13.0 with foss/2022b
and foss/2023a
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure. Given I found and worked around the Bazel issue in 2022b+ we could also skip this to avoid possible confusion/mixup with module versions.
Closing since we have this for 2022b & 2023a |
(created using
eb --new-pr
)