Skip to content

Commit

Permalink
Pin etils to fix error with tfds.load (#1086)
Browse files Browse the repository at this point in the history
Signed-off-by: ashors1 <ashors@nvidia.com>
  • Loading branch information
ashors1 authored Oct 8, 2024
1 parent 032e978 commit d1ff3c8
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .github/container/Dockerfile.pax.amd64
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ for src in ${SRC_PATH_PAXML} ${SRC_PATH_PRAXIS}; do
pushd ${src}
sed -i "s| @ git+https://github.com/google/flax||g" requirements.in
sed -i "s| @ git+https://github.com/google/jax||g" requirements.in
## we pin etils because newer etils versions are not compatible with the
## version of TFDS required by Pax
sed -i "s/etils/etils==1.7.0/g" requirements.in
if git diff --quiet; then
echo "URL specs no longer present in select dependencies for ${src}"
exit 1
Expand Down

0 comments on commit d1ff3c8

Please sign in to comment.