Skip to content

feat: add bigframes.bigquery.st_simplify - #2210

Merged
tswast merged 6 commits into
mainfrom
feat-st-simplify
Oct 31, 2025
Merged

feat: add bigframes.bigquery.st_simplify#2210
tswast merged 6 commits into
mainfrom
feat-st-simplify

Conversation

@tswast

Copy link
Copy Markdown
Collaborator

Towards the code sample in #2200 🦕

@tswast
tswast requested review from a team and GarrettWuOctober 30, 2025 18:43
@product-auto-labelproduct-auto-labelBot added size: m Pull request size is medium. api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. labels Oct 30, 2025
@tswast
tswast requested review from jialuoo and removed request for GarrettWuOctober 30, 2025 19:11
@tswast

Copy link
Copy Markdown
CollaboratorAuthor

@jialuoo could you take a look? #2200 was also assigned to you, but I think it's important to do this one first.


Raises:
NotImplementedError:
GeoSeries.area is not supported. Use bigframes.bigquery.st_area(series), instead.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it 'st_simplify' here instead of 'st_area'?

`NotImplementedError` with a helpful message.
`NotImplementedError` with a helpful message. Likewise, if a
required parameter takes a value in terms of the coordinate
system, but BigQuery uses meters, rais a `NotImplementedError`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

typo here: 'rais'

@tswast
tswast requested a review from jialuooOctober 31, 2025 16:12
@tswast

Copy link
Copy Markdown
CollaboratorAuthor

doctest failure is FAILED bigframes/bigquery/_operations/ai.py::bigframes.bigquery._operations.ai.generate likely a flake.

@tswast
tswast merged commit ecee2bc into mainOct 31, 2025
19 of 25 checks passed
@tswast
tswast deleted the feat-st-simplify branch October 31, 2025 17:13
@release-pleaserelease-pleaseBot mentioned this pull request Oct 31, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigqueryIssues related to the googleapis/python-bigquery-dataframes API.size: mPull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tswast@jialuoo@liujiashang