Skip to content

Commit cffe89d

Browse files
author
Phil Varner
authored
deprecate grid_geo* aggs, add centroid_geo* aggs, add geometry_geo* aggs (#702)
1 parent 8b9c3eb commit cffe89d

File tree

10 files changed

+1813
-1221
lines changed

10 files changed

+1813
-1221
lines changed

.npm-upgrade.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
"reason": ""
1414
},
1515
"ava": {
16-
"versions": "6.0.1",
17-
"reason": ""
16+
"versions": "6",
17+
"reason": "breaks test running"
1818
}
1919
}
2020
}

CHANGELOG.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,20 @@ All notable changes to this project will be documented in this file.
55
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
66
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
77

8+
## [Unreleased] - TBD
9+
10+
## Changed
11+
12+
- Deprecated `grid_geohex_frequency`, `grid_geohash_frequency`, and
13+
`grid_geotile_frequency` aggregations in favor of new `centroid_geohash_grid_frequency`,
14+
`centroid_geohex_grid_frequency`, and `centroid_geotile_grid_frequency` aggregations
15+
16+
## Added
17+
18+
- Added `geometry_geohash_grid_frequency` and `geometry_geotile_grid_frequency` that
19+
aggregate over the geometry of each Item rather than the centroid. Note that the geohex aggregation `geometry_geohex_grid_frequency` is **not** implemented, as OpenSearch 2.11
20+
does not yet support geohex_grid aggregations over geo_shape fields.
21+
822
## [3.5.0] - 2024-01-19
923

1024
### Fixed

README.md

Lines changed: 22 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -932,17 +932,27 @@ the collection for which the aggregations are available, e.g.:
932932
"frequency_distribution_data_type": "string"
933933
},
934934
{
935-
"name": "grid_geohex_frequency",
935+
"name": "centroid_geohash_grid_frequency",
936936
"data_type": "frequency_distribution",
937937
"frequency_distribution_data_type": "string"
938938
},
939939
{
940-
"name": "grid_geohash_frequency",
940+
"name": "centroid_geohex_grid_frequency",
941941
"data_type": "frequency_distribution",
942942
"frequency_distribution_data_type": "string"
943943
},
944944
{
945-
"name": "grid_geotile_frequency",
945+
"name": "centroid_geotile_grid_frequency",
946+
"data_type": "frequency_distribution",
947+
"frequency_distribution_data_type": "string"
948+
},
949+
{
950+
"name": "geometry_geohash_grid_frequency",
951+
"data_type": "frequency_distribution",
952+
"frequency_distribution_data_type": "string"
953+
},
954+
{
955+
"name": "geometry_geotile_grid_frequency",
946956
"data_type": "frequency_distribution",
947957
"frequency_distribution_data_type": "string"
948958
}
@@ -958,13 +968,18 @@ Available aggregations are:
958968
- datetime_frequency (Item.Properties.datetime, monthly interval)
959969
- datetime_min (earliest Item.Properties.datetime)
960970
- datetime_max (latest Item.Properties.datetime)
961-
- grid_code_frequency (Item.Properties.grid:code)
962971
- sun_elevation_frequency (Item.Properties.view:sun_elevation)
963972
- sun_azimuth_frequency (Item.Properties.view:sun_azimuth)
964973
- off_nadir_frequency (Item.Properties.view:off_nadir)
965-
- grid_geohex_frequency ([GeoHex grid](https://opensearch.org/docs/2.4/opensearch/geohexgrid-agg/) on Item.Properties.proj:centroid)
966-
- grid_geohash_frequency ([geohash grid](https://opensearch.org/docs/2.4/opensearch/bucket-agg/#geo_distance-geohash_grid) on Item.Properties.proj:centroid)
967-
- grid_geotile_frequency (geotile on Item.Properties.proj:centroid)
974+
- grid_code_frequency (Item.Properties.grid:code)
975+
- grid_geohash_frequency ([geohash grid](https://opensearch.org/docs/latest/aggregations/bucket/geohash-grid/) on Item.Properties.proj:centroid) (Deprecated)
976+
- grid_geohex_frequency ([geohex grid](https://opensearch.org/docs/latest/aggregations/bucket/geohex-grid/) on Item.Properties.proj:centroid) (Deprecated)
977+
- grid_geotile_frequency ([geotile grid](https://opensearch.org/docs/latest/aggregations/bucket/geotile-grid/) on Item.Properties.proj:centroid) (Deprecated)
978+
- centroid_geohash_grid_frequency ([geohash grid](https://opensearch.org/docs/latest/aggregations/bucket/geohash-grid/) on Item.Properties.proj:centroid)
979+
- centroid_geohex_grid_frequency ([geohex grid](https://opensearch.org/docs/latest/aggregations/bucket/geohex-grid/) on Item.Properties.proj:centroid)
980+
- centroid_geotile_grid_frequency (geotile on Item.Properties.proj:centroid)
981+
- geometry_geohash_grid_frequency ([geohash grid](https://opensearch.org/docs/latest/aggregations/bucket/geohash-grid/) on Item.geometry)
982+
- geometry_geotile_grid_frequency ([geotile grid](https://opensearch.org/docs/latest/aggregations/bucket/geotile-grid/) on Item.geometry)
968983

969984
## Ingesting Data
970985

0 commit comments

Comments
 (0)