Skip to content

chore(bigframes): clean up the the AI package#17010

Draft
sycai wants to merge 1 commit intomainfrom
sycai_ai_cleanup
Draft

chore(bigframes): clean up the the AI package#17010
sycai wants to merge 1 commit intomainfrom
sycai_ai_cleanup

Conversation

@sycai
Copy link
Copy Markdown
Contributor

@sycai sycai commented May 9, 2026

The contents of the change are:

  • Skipped all the doctests for the AI package.
  • Fixed Sphinx format error: single backticks are replaced with double backticks.
  • Removed parameter verification to keep the client lean.
  • Removed pre-GA notes as these functions are all GA'ed
  • Fixed format of some other files .

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation and refactors validation logic within the AI operations module. Key changes include adding doctest skip markers to examples, standardizing docstring formatting with double backticks for literals, and removing 'Pre-GA' notice blocks. Additionally, manual argument validation for the embed and similarity functions has been removed along with their corresponding unit tests, and imports have been reorganized across several files. I have no feedback to provide as there were no review comments to assess.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant