Skip to content

docs: add a visualization notebook to BigFrame samples #1675

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
May 1, 2025
Merged

Conversation

sycai
Copy link
Contributor

@sycai sycai commented Apr 30, 2025

Also fixed broken links in the Covid notebook.

@product-auto-label product-auto-label bot added the size: xl Pull request size is extra large. label Apr 30, 2025
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@product-auto-label product-auto-label bot added api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. samples Issues that are directly related to samples. labels Apr 30, 2025
@sycai sycai requested a review from chelsea-lin April 30, 2025 19:03
@sycai sycai marked this pull request as ready for review April 30, 2025 19:08
@sycai sycai requested review from a team as code owners April 30, 2025 19:08
Copy link
Contributor

@chelsea-lin chelsea-lin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The visualization examples in the notebooks are excellent – thanks!
To further demonstrate the robustness of our native visualization tools, could we include examples of more complex plot configurations? This would highlight that our tools support the full range of plotting functionalities available in pandas.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To indicate this notebook is the recommended starting point, perhaps we could name it 'Tutorial' or 'Examples.' This would differentiate it from notebooks like 'bq_dataframes_covid_line_graphs.ipynb' which might be for more specific use cases.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SG. Renamed to "tutorial.py"

@sycai
Copy link
Contributor Author

sycai commented Apr 30, 2025

The visualization examples in the notebooks are excellent – thanks! To further demonstrate the robustness of our native visualization tools, could we include examples of more complex plot configurations? This would highlight that our tools support the full range of plotting functionalities available in pandas.

@chelsea-lin I added an enhanced version of taxi scatter plot

@sycai sycai requested a review from chelsea-lin April 30, 2025 22:10
@@ -0,0 +1,1470 @@
{
Copy link
Contributor

@chelsea-lin chelsea-lin Apr 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe starting another section with title called "Advanced Plotting with Pandas and Matplotlib" to highlight the ability. The scatter method is just one of examples?


Reply via ReviewNB

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure. Let's add more examples in the future.

Copy link
Contributor

@chelsea-lin chelsea-lin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with one nit comment. Thanks!

@sycai sycai enabled auto-merge (squash) May 1, 2025 17:51
@sycai sycai merged commit ee062bf into main May 1, 2025
19 of 24 checks passed
@sycai sycai deleted the sycai_vis branch May 1, 2025 22:29
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. samples Issues that are directly related to samples. size: xl Pull request size is extra large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants