Skip to content

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

Open
sycai wants to merge 1 commit into
mainfrom
sycai_ai_cleanup
Open

chore(bigframes): clean up the the AI package#17010
sycai wants to merge 1 commit into
mainfrom
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.

@sycai sycai requested review from TrevorBergeron and tswast May 11, 2026 17:06
@sycai sycai marked this pull request as ready for review May 11, 2026 17:06
@sycai sycai requested review from a team as code owners May 11, 2026 17:06
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