Skip to content

Commit

Permalink
remove time_range from groupby_attrs in catalog_builder, example json…
Browse files Browse the repository at this point in the history
…, and util/catalog.py (NOAA-GFDL#652)
  • Loading branch information
wrongkindofdoctor authored Aug 6, 2024
1 parent 27e2a3d commit aeacf6a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 15 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -163,23 +163,13 @@
"table_id",
"grid_label",
"realm",
"variant_label",
"time_range"
"variant_label"
],
"aggregations": [
{
"type": "union",
"attribute_name": "variable_id",
"options": {}
},
{
"type": "join_existing",
"attribute_name": "time_range",
"options": {
"dim": "time",
"coords": "minimal",
"compat": "override"
}
}
]
},
Expand Down
3 changes: 1 addition & 2 deletions src/util/catalog.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,7 @@ def define_pp_catalog_assets(config, cat_file_name: str) -> dict:
"grid_label",
"realm",
"chunk_freq",
"variant_label",
"time_range"
"variant_label"
],
"aggregations": [
{
Expand Down
3 changes: 1 addition & 2 deletions tools/catalog_builder/catalog_builder.py
Original file line number Diff line number Diff line change
Expand Up @@ -302,8 +302,7 @@ def __init__(self):
'table_id',
'grid_label',
'realm',
'variant_label',
'time_range'
'variant_label'
] # attributes to group by when reading
# in variables using intake-esm
self.xarray_aggregations = [
Expand Down

0 comments on commit aeacf6a

Please sign in to comment.