Skip to content

Commit bd0bce1

Browse files
iindykcopybara-github
authored andcommitted
Testing pytest.
PiperOrigin-RevId: 886439501
1 parent 146edf3 commit bd0bce1

7 files changed

Lines changed: 7 additions & 250 deletions

File tree

.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,6 @@ jobs:
1616
id-token: write
1717
with:
1818
pypi_project_url: ''
19-
run_tests_with_bazel: true
19+
run_tests_with_bazel: false
2020
is_nightly: false
2121
is_ci_tests: true

grain/_src/python/checkpoint/handler_test.py

Lines changed: 0 additions & 130 deletions
This file was deleted.

grain/_src/python/checkpoint/orbax_integration_test.py

Lines changed: 0 additions & 110 deletions
This file was deleted.

grain/_src/python/dataset/sources/BUILD

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,6 @@ py_library(
2929
name = "tfrecord_dataset",
3030
srcs = ["tfrecord_dataset.py"],
3131
srcs_version = "PY3",
32-
visibility = [
33-
"//third_party/py/grain:internal",
34-
"//third_party/py/maxtext:__pkg__",
35-
],
3632
deps = ["//grain/_src/python/dataset"],
3733
)
3834

grain/oss/build_whl.sh

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ main() {
2121
# Enable host OS specific configs. For instance, "build:linux" will be used
2222
# automatically when building on Linux.
2323
write_to_bazelrc "build --enable_platform_specific_config"
24-
write_to_bazelrc "build --verbose_failures"
2524
# Bazel 7.0.0 no longer supports dynamic symbol lookup on macOS. To resolve
2625
# undefined symbol errors in macOS arm64 builds, explicitly add the necessary
2726
# linker flags until dependencies are well defined. See
@@ -133,8 +132,10 @@ main() {
133132
$PYTHON_BIN -m pip install --force ${OUTPUT_DIR}/all_dist/grain*.whl
134133
$PYTHON_BIN -m pip install -r test_requirements.in --only-binary pyarrow
135134
# TF does not have Python 3.14 support yet.
135+
# Once majority of our users are on TF >= 2.21, we should update the
136+
# protobuf dependency and bump the smoke test to TF >= 2.21.
136137
if (( "${PYTHON_MINOR_VERSION}" < 14 )); then
137-
$PYTHON_BIN -m pip install tensorflow --only-binary h5py
138+
$PYTHON_BIN -m pip install "tensorflow<2.21" --only-binary h5py
138139
fi
139140

140141
pushd "${OUTPUT_DIR}/all_dist"

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ dependencies = [
1616
"cloudpickle",
1717
"etils[epath,epy]",
1818
"numpy",
19-
"protobuf>=5.28.3",
19+
"protobuf>=5.28.3,<7",
2020
]
2121
readme = "README.md"
2222
license = { file = "LICENSE" }

test_requirements.in

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,10 @@ array-record
2020
absl-py
2121
etils[epath,epy]
2222
cloudpickle
23-
jax
23+
jax<0.9
2424
numpy
2525
attrs
26-
pyarrow
26+
pyarrow<23.0 # This should be update with protobuf dependency update.
2727
pytest
2828
parameterized
2929
typing_extensions

0 commit comments

Comments
 (0)