googleapis · chalmerlowe · Apr 10, 2026 · Apr 10, 2026 · Apr 10, 2026 · Apr 10, 2026
@@ -43,7 +43,8 @@ run_package_test() {
   local PROJECT_ID
   local GOOGLE_APPLICATION_CREDENTIALS
   local NOX_FILE
-  local NOX_SESSION
+  # Inherit NOX_SESSION from environment to allow configs (like prerelease.cfg) to pass it in
+  local NOX_SESSION="${NOX_SESSION}"
 
   echo "------------------------------------------------------------"
   echo "Configuring environment for: ${package_name}"
@@ -66,7 +67,8 @@ run_package_test() {
       PROJECT_ID=$(cat "${KOKORO_GFILE_DIR}/project-id.json")
       GOOGLE_APPLICATION_CREDENTIALS="${KOKORO_GFILE_DIR}/service-account.json"
       NOX_FILE="noxfile.py"
-      NOX_SESSION="system-3.12"
+      # Use inherited NOX_SESSION if set, otherwise fallback to system-3.12
+      NOX_SESSION="${NOX_SESSION:-system-3.12}"
       ;;
   esac
 

@@ -2796,6 +2796,7 @@ def _is_monotonic(
                 )
                 block = block.drop_columns([equal_monotonic_id, strict_monotonic_id])
 
+        assert last_result_id is not None
         block, monotonic_result_id = block.apply_binary_op(
             last_result_id,
             last_notna_id,

@@ -528,8 +528,9 @@ def _(
     column: ibis_types.Column,
     window=None,
 ) -> ibis_types.Value:
+    # Ibis FirstNonNullValue expects Value[Any, Columnar], Mypy struggles to see Column as compatible.
     return _apply_window_if_present(
-        ibis_ops.FirstNonNullValue(column).to_expr(),
+        ibis_ops.FirstNonNullValue(column).to_expr(),  # type: ignore[arg-type]
         window,  # type: ignore
     )
 
@@ -549,8 +550,9 @@ def _(
     column: ibis_types.Column,
     window=None,
 ) -> ibis_types.Value:
+    # Ibis LastNonNullValue expects Value[Any, Columnar], Mypy struggles to see Column as compatible.
     return _apply_window_if_present(
-        ibis_ops.LastNonNullValue(column).to_expr(),
+        ibis_ops.LastNonNullValue(column).to_expr(),  # type: ignore[arg-type]
         window,  # type: ignore
     )
 
@@ -803,8 +805,9 @@ def _to_ibis_boundary(
 ) -> Optional[ibis_expr_window.WindowBoundary]:
     if boundary is None:
         return None
+    # WindowBoundary expects Value[Any, Any], ibis_types.literal returns Scalar which Mypy doesn't see as compatible.
     return ibis_expr_window.WindowBoundary(
-        abs(boundary),
+        ibis_types.literal(boundary if boundary >= 0 else -boundary),  # type: ignore[arg-type]
         preceding=boundary <= 0,  # type:ignore
     )
 

@@ -49,8 +49,8 @@ def compile_sql(request: configs.CompileRequest) -> configs.CompileResult:
         # Can only pullup slice if we are doing ORDER BY in outermost SELECT
         # Need to do this before replacing unsupported ops, as that will rewrite slice ops
         result_node = rewrites.pull_up_limits(result_node)
-    result_node = _replace_unsupported_ops(result_node)
-    result_node = result_node.bottom_up(rewrites.simplify_join)
+    result_node = cast(nodes.ResultNode, _replace_unsupported_ops(result_node))
+    result_node = cast(nodes.ResultNode, result_node.bottom_up(rewrites.simplify_join))
     # prune before pulling up order to avoid unnnecessary row_number() ops
     result_node = cast(nodes.ResultNode, rewrites.column_pruning(result_node))
     result_node = rewrites.defer_order(

@@ -182,7 +182,7 @@ def st_buffer(
 
 @ibis_udf.scalar.builtin
 def st_distance(
-    a: ibis_dtypes.geography, b: ibis_dtypes.geography, use_spheroid: bool
+    a: ibis_dtypes.geography, b: ibis_dtypes.geography, use_spheroid: bool  # type: ignore
 ) -> ibis_dtypes.float:  # type: ignore
     """Convert string to geography."""
 

@@ -2168,9 +2168,12 @@ def obj_make_ref_json(objectref_json: ibis_dtypes.JSON) -> _OBJ_REF_IBIS_DTYPE:
 
 
 @ibis_udf.scalar.builtin(name="OBJ.GET_ACCESS_URL")
-def obj_get_access_url(
-    obj_ref: _OBJ_REF_IBIS_DTYPE, mode: ibis_dtypes.String
-) -> ibis_dtypes.JSON:  # type: ignore
+# Stub for BigQuery UDF, empty body is intentional.
+# _OBJ_REF_IBIS_DTYPE is a variable holding a type, Mypy complains about it being used as type hint.
+def obj_get_access_url(  # type: ignore[empty-body]
+    obj_ref: _OBJ_REF_IBIS_DTYPE,  # type: ignore[valid-type]
+    mode: ibis_dtypes.String
+) -> ibis_dtypes.JSON:
     """Get access url (as ObjectRefRumtime JSON) from ObjectRef."""
 
 

@@ -53,8 +53,8 @@ def compile_sql(request: configs.CompileRequest) -> configs.CompileResult:
         # Can only pullup slice if we are doing ORDER BY in outermost SELECT
         # Need to do this before replacing unsupported ops, as that will rewrite slice ops
         result_node = rewrite.pull_up_limits(result_node)
-    result_node = _replace_unsupported_ops(result_node)
-    result_node = result_node.bottom_up(rewrite.simplify_join)
+    result_node = typing.cast(nodes.ResultNode, _replace_unsupported_ops(result_node))
+    result_node = typing.cast(nodes.ResultNode, result_node.bottom_up(rewrite.simplify_join))
     # prune before pulling up order to avoid unnnecessary row_number() ops
     result_node = typing.cast(nodes.ResultNode, rewrite.column_pruning(result_node))
     result_node = rewrite.defer_order(

@@ -243,7 +243,7 @@ def factor_aggregation(root: nodes.ColumnDef) -> FactoredAggregation:
     }
 
     root_scalar_expr = nodes.ColumnDef(
-        sub_expressions(root.expression, agg_outputs_dict),
+        sub_expressions(root.expression, cast(Mapping[expression.Expression, expression.Expression], agg_outputs_dict)),
         root.id,  # type: ignore
     )
 

@@ -33,6 +33,7 @@
 
 import bigframes.core.schema as schemata
 import bigframes.dtypes
+from bigframes.core import identifiers
 from bigframes.core import pyarrow_utils
 
 
@@ -155,7 +156,7 @@ def to_arrow(
             return schema, batches
 
     def is_nullable(self, column_id: identifiers.ColumnId) -> bool:
-        return self.data.column(column_id).null_count > 0
+        return self.data.column(column_id.name).null_count > 0
 
     def to_pyarrow_table(
         self,

@@ -674,7 +674,7 @@ def fields(self) -> Sequence[Field]:
             Field(
                 col_id,
                 self.local_data_source.schema.get_type(source_id),
-                nullable=self.local_data_source.is_nullable(source_id),
+                nullable=self.local_data_source.is_nullable(identifiers.ColumnId(source_id)),
             )
             for col_id, source_id in self.scan_list.items
         )

@@ -291,8 +291,9 @@ def _extract_ctes_to_with_expr(
         root.top_down(lambda x: mapping.get(x, x)),
         cte_names,
         tuple(
-            cte_node.child.top_down(lambda x: mapping.get(x, x))
-            for cte_node in topological_ctes  # type: ignore
+            # Mypy loses context that cte_node is a CteNode with a child attribute, despite the isinstance filter above.
+            cte_node.child.top_down(lambda x: mapping.get(x, x))  # type: ignore[attr-defined]
+            for cte_node in topological_ctes
         ),
     )
 

@@ -3926,12 +3926,13 @@ def round(self, decimals: Union[int, dict[Hashable, int]] = 0) -> DataFrame:
                 bigframes.dtypes.BOOL_DTYPE
             }:
                 if is_mapping:
-                    if label in decimals:  # type: ignore
+                    decimals_dict = typing.cast(dict[typing.Hashable, int], decimals)
+                    if label in decimals_dict:
                         exprs.append(
                             ops.round_op.as_expr(
                                 col_id,
                                 ex.const(
-                                    decimals[label],
+                                    decimals_dict[label],
                                     dtype=bigframes.dtypes.INT_DTYPE,  # type: ignore
                                 ),
                             )
@@ -4447,8 +4448,8 @@ def to_latex(
     ) -> str | None:
         return self.to_pandas(allow_large_results=allow_large_results).to_latex(
             buf,
-            columns=columns,
-            header=header,
+            columns=typing.cast(typing.Optional[list[str]], columns),
+            header=typing.cast(typing.Union[bool, list[str]], header),
             index=index,
             **kwargs,  # type: ignore
         )

@@ -2308,9 +2308,10 @@ def to_json(
             )
         else:
             pd_series = self.to_pandas(allow_large_results=allow_large_results)
+            # Pandas Series.to_json only supports a subset of orients, but bigframes Series.to_json allows all of them.
             return pd_series.to_json(
                 path_or_buf=path_or_buf,
-                orient=orient,
+                orient=orient,  # type: ignore[arg-type]
                 lines=lines,
                 index=index,  # type: ignore
             )

@@ -98,9 +98,10 @@ def _extract_location_from_catalog_extension_data(data):
 
 
 class SchemaVisitor(pyiceberg.schema.SchemaVisitorPerPrimitiveType[bq.SchemaField]):
-    def schema(
+    # Override returns a tuple of fields instead of a single field, violating supertype signature but intentional for this visitor.
+    def schema(  # type: ignore[override]
         self, schema: pyiceberg.schema.Schema, struct_result: bq.SchemaField
-    ) -> tuple[bq.SchemaField, ...]:  # type: ignore
+    ) -> tuple[bq.SchemaField, ...]:
         return tuple(f for f in struct_result.fields)
 
     def struct(