- Notifications
You must be signed in to change notification settings - Fork63
feat: add bigframes.bigquery.st_regionstats to join raster data from Earth Engine#2228
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
snippet-botbot commentedNov 4, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Here is the summary of changes. You are about to add 1 region tag.
This comment is generated bysnippet-bot.
|
tswast commentedNov 4, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Python 3.9 failures look real. Need to bump the minimum SQLGlot version to support JSON literals, is my guess. Edit: Done! I had to bump it even a bit further to avoid a bug with ltrim/rtrim. |
tswast commentedNov 4, 2025
Internal cl/828117205 to hopefully fix the permissions error on the code sample. |
| defst_regionstats( | ||
| geography:Union[bigframes.series.Series,bigframes.geopandas.GeoSeries], | ||
| raster_id:str, | ||
| band:Optional[str]=None, |
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.
To be consistent, we may want to make the optional args as kwargs.
def st_length( series: Union[bigframes.series.Series, bigframes.geopandas.GeoSeries], *, use_spheroid: bool = False,)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.
Thanks. I'm okay either way with this one. In my testing, BigQuery SQL actually accepts the optional parameters to ST_REGIONSTATS as either keyword arguments or positional, so the Python default seems sensible here.
10ec52f intomainUh oh!
There was an error while loading.Please reload this page.
🤖 I have created a release *beep* *boop*---##[2.29.0](v2.28.0...v2.29.0)(2025-11-10)### Features* Add bigframes.bigquery.st_regionstats to join raster data from EarthEngine([#2228](#2228))([10ec52f](10ec52f))* Add DataFrame.resample and Series.resample([#2213](#2213))([c9ca02c](c9ca02c))* SQL Cell no longer escapes formatted string values([#2245](#2245))([d2d38f9](d2d38f9))* Support left_index and right_index for merge([#2220](#2220))([da9ba26](da9ba26))### Bug Fixes* Correctly iterate over null struct values in ManagedArrowTable([#2209](#2209))([12e04d5](12e04d5))* Simplify UnsupportedTypeError message([#2212](#2212))([6c9a18d](6c9a18d))* Support results with STRUCT and ARRAY columns containing JSONsubfields in `to_pandas_batches()`([#2216](#2216))([3d8b17f](3d8b17f))### Documentation* Switch API reference docs to pydata theme([#2237](#2237))([9b86dcf](9b86dcf))* Update notebook for JSON subfields support in to_pandas_batches()([#2138](#2138))([5663d2a](5663d2a))---This PR was generated with [ReleasePlease](https://github.com/googleapis/release-please). See[documentation](https://github.com/googleapis/release-please#release-please).Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Towards internal issue b/457755215 🦕