Skip to content

Conversation

@shuoweil
Copy link
Contributor

THIS PR is not ready to review. I need this pr to test it on colab notebooks.

This PR fixes a crash (IndexError: tuple index out of range) that occurred when performing unpivot operations, such as melt, stack, or summarize, on empty or filtered DataFrames. This issue was frequently encountered by users in notebook workflows, particularly when using SQL Cells and Viz Cells on result sets with zero rows.

Fixes #<452681068> 🦕

@shuoweil shuoweil self-assigned this Dec 23, 2025
@shuoweil shuoweil requested review from a team as code owners December 23, 2025 22:02
@shuoweil shuoweil requested a review from jialuoo December 23, 2025 22:02
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. labels Dec 23, 2025
@shuoweil shuoweil marked this pull request as draft December 23, 2025 22:02
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: m Pull request size is medium. labels Jan 7, 2026
@shuoweil shuoweil removed the request for review from jialuoo January 7, 2026 05:48
@shuoweil shuoweil force-pushed the shuowei-anywidget-fix-empty-index branch from bcd9cb7 to 7ad3a04 Compare January 7, 2026 06:22
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Jan 7, 2026
- Replaced Python-based row explosion with optimized PyArrow computation for nested arrays.
- Cleaned up comments in  to strictly adhere to Google Python Style Guide (focused on 'why', removed redundant 'what').
- Renamed variable  to  for clarity.
- Verified changes with Python unit tests and JavaScript frontend tests.
@shuoweil shuoweil force-pushed the shuowei-anywidget-fix-empty-index branch from f4b3afd to 210cb2f Compare January 12, 2026 19:32
@shuoweil shuoweil force-pushed the shuowei-anywidget-fix-empty-index branch 3 times, most recently from 3c8efcc to ac2b717 Compare January 14, 2026 01:35
@shuoweil shuoweil force-pushed the shuowei-anywidget-fix-empty-index branch from ac2b717 to 28bbedd Compare January 14, 2026 01:50
@shuoweil shuoweil force-pushed the shuowei-anywidget-fix-empty-index branch from 28bbedd to 32985d8 Compare January 14, 2026 02:11
@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. and removed size: m Pull request size is medium. labels Jan 14, 2026
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. size: xl Pull request size is extra large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant