From c4024681d0dce010bf80dd18e5071f4858f28495 Mon Sep 17 00:00:00 2001 From: chalmer lowe Date: Mon, 24 Nov 2025 18:05:27 -0500 Subject: [PATCH 1/5] chore(python): removes a filter put in place due to a dependency issue with pyarrow --- .github/workflows/unittest.yml | 6 ------ 1 file changed, 6 deletions(-) diff --git a/.github/workflows/unittest.yml b/.github/workflows/unittest.yml index f6b92547e..550724076 100644 --- a/.github/workflows/unittest.yml +++ b/.github/workflows/unittest.yml @@ -21,12 +21,6 @@ jobs: python -m pip install --upgrade setuptools pip wheel python -m pip install nox - name: Run unit tests - - # TODO (https://b.corp.google.com/issues/450370502) 3.14 is not yet supported by pyarrow. See - # https://github.com/googleapis/google-cloud-python/issues/14686 - # https://github.com/apache/arrow/issues/47438 - # Reinstate running tests with 3.14 once this bug is fixed - if: matrix.python != '3.14' env: COVERAGE_FILE: .coverage-${{ matrix.python }} run: | From 3266732d4e6a208fd2e10ecc41a2da02fca5fee0 Mon Sep 17 00:00:00 2001 From: chalmer lowe Date: Mon, 24 Nov 2025 18:17:23 -0500 Subject: [PATCH 2/5] temporarily filter out geopandas to avoid failing a different reason for failing testJ --- noxfile.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/noxfile.py b/noxfile.py index 194e7ce8f..9d85b8147 100644 --- a/noxfile.py +++ b/noxfile.py @@ -110,7 +110,10 @@ def default(session, install_extras=True): # that logic (and the associated tests) we avoid installing the [ipython] extra # which has a downstream effect of then avoiding installing bigquery_magics. if install_extras and session.python == UNIT_TEST_PYTHON_VERSIONS[0]: - install_target = ".[bqstorage,pandas,ipywidgets,geopandas,matplotlib,tqdm,opentelemetry,bigquery_v2]" + # install_target = ".[bqstorage,pandas,ipywidgets,geopandas,matplotlib,tqdm,opentelemetry,bigquery_v2]" + install_target = ( + ".[bqstorage,pandas,ipywidgets,matplotlib,tqdm,opentelemetry,bigquery_v2]" + ) elif install_extras: # run against all other UNIT_TEST_PYTHON_VERSIONS install_target = ".[all]" else: From b47828471af0c440488ad1c6b6c4339447b3cf41 Mon Sep 17 00:00:00 2001 From: chalmer lowe Date: Mon, 24 Nov 2025 18:23:43 -0500 Subject: [PATCH 3/5] temporarily filter out geopandas to avoid failing a different reason for failing test redux --- noxfile.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/noxfile.py b/noxfile.py index 9d85b8147..4171adf81 100644 --- a/noxfile.py +++ b/noxfile.py @@ -110,12 +110,11 @@ def default(session, install_extras=True): # that logic (and the associated tests) we avoid installing the [ipython] extra # which has a downstream effect of then avoiding installing bigquery_magics. if install_extras and session.python == UNIT_TEST_PYTHON_VERSIONS[0]: - # install_target = ".[bqstorage,pandas,ipywidgets,geopandas,matplotlib,tqdm,opentelemetry,bigquery_v2]" + install_target = ".[bqstorage,pandas,ipywidgets,geopandas,matplotlib,tqdm,opentelemetry,bigquery_v2]" + elif install_extras: # run against all other UNIT_TEST_PYTHON_VERSIONS install_target = ( ".[bqstorage,pandas,ipywidgets,matplotlib,tqdm,opentelemetry,bigquery_v2]" ) - elif install_extras: # run against all other UNIT_TEST_PYTHON_VERSIONS - install_target = ".[all]" else: install_target = "." session.install("-e", install_target, "-c", constraints_path) From 9afe16ff039daaa3b6d803ce17d8e6170d6cea14 Mon Sep 17 00:00:00 2001 From: Chalmer Lowe Date: Mon, 24 Nov 2025 19:34:24 -0500 Subject: [PATCH 4/5] Apply suggestion from @chalmerlowe --- noxfile.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/noxfile.py b/noxfile.py index 4171adf81..a5a59839e 100644 --- a/noxfile.py +++ b/noxfile.py @@ -112,9 +112,6 @@ def default(session, install_extras=True): if install_extras and session.python == UNIT_TEST_PYTHON_VERSIONS[0]: install_target = ".[bqstorage,pandas,ipywidgets,geopandas,matplotlib,tqdm,opentelemetry,bigquery_v2]" elif install_extras: # run against all other UNIT_TEST_PYTHON_VERSIONS - install_target = ( - ".[bqstorage,pandas,ipywidgets,matplotlib,tqdm,opentelemetry,bigquery_v2]" - ) else: install_target = "." session.install("-e", install_target, "-c", constraints_path) From 7a8a6e1fbc16bc509d5cc408229ab93b873f18c3 Mon Sep 17 00:00:00 2001 From: chalmer lowe Date: Mon, 24 Nov 2025 19:38:57 -0500 Subject: [PATCH 5/5] removes temp edit to allows testing in CI system --- noxfile.py | 1 + 1 file changed, 1 insertion(+) diff --git a/noxfile.py b/noxfile.py index a5a59839e..194e7ce8f 100644 --- a/noxfile.py +++ b/noxfile.py @@ -112,6 +112,7 @@ def default(session, install_extras=True): if install_extras and session.python == UNIT_TEST_PYTHON_VERSIONS[0]: install_target = ".[bqstorage,pandas,ipywidgets,geopandas,matplotlib,tqdm,opentelemetry,bigquery_v2]" elif install_extras: # run against all other UNIT_TEST_PYTHON_VERSIONS + install_target = ".[all]" else: install_target = "." session.install("-e", install_target, "-c", constraints_path)