- Notifications
You must be signed in to change notification settings - Fork63
docs: update bq_dataframes_llm_output_schema.ipynb#2004
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
* Cleaned up language and structure.* Added set up and clean up sections.
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered byReviewNB |
| " </a>\n", | ||
| " </td>\n", | ||
| " <td>\n", | ||
| " <a href=\"https://github.com/googleapis/python-bigquery-dataframes/blob/main/notebooks/generative_ai/bq_dataframes_llm_output_schema.ipynb\">\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Maybe we should use a Vertext AI Workbench link instead of a Github link?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I had added that option in, and I just removed it again. It isn't really needed, since we want users to open the notebook in BigQuery.
316ba9f intogoogleapis:mainUh oh!
There was an error while loading.Please reload this page.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
Fixes#2000 🦕
Unit tests for Python 3.12 passed, notebook works when tested in the console.