-
Notifications
You must be signed in to change notification settings - Fork 55
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
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 by ReviewNB |
@@ -40,10 +40,16 @@ | |||
" View on GitHub\n", | |||
" </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.
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.