From aeacf6a3bfbb8b56d6104df60081f712565e3d40 Mon Sep 17 00:00:00 2001
From: Jess <20195932+wrongkindofdoctor@users.noreply.github.com>
Date: Tue, 6 Aug 2024 15:23:53 -0400
Subject: [PATCH] remove time_range from groupby_attrs in catalog_builder,
 example json, and util/catalog.py (#652)

---
 .../esm_catalog_CMIP_synthetic_r1i1p1f1_gr1.json     | 12 +-----------
 src/util/catalog.py                                  |  3 +--
 tools/catalog_builder/catalog_builder.py             |  3 +--
 3 files changed, 3 insertions(+), 15 deletions(-)

diff --git a/diagnostics/example_multicase/esm_catalog_CMIP_synthetic_r1i1p1f1_gr1.json b/diagnostics/example_multicase/esm_catalog_CMIP_synthetic_r1i1p1f1_gr1.json
index b9029f0f4..c62245645 100644
--- a/diagnostics/example_multicase/esm_catalog_CMIP_synthetic_r1i1p1f1_gr1.json
+++ b/diagnostics/example_multicase/esm_catalog_CMIP_synthetic_r1i1p1f1_gr1.json
@@ -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"
-        }
       }
     ]
   },
diff --git a/src/util/catalog.py b/src/util/catalog.py
index e05e06ac2..268f75b3c 100644
--- a/src/util/catalog.py
+++ b/src/util/catalog.py
@@ -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": [
             {
diff --git a/tools/catalog_builder/catalog_builder.py b/tools/catalog_builder/catalog_builder.py
index f2e944df0..cfe95644d 100644
--- a/tools/catalog_builder/catalog_builder.py
+++ b/tools/catalog_builder/catalog_builder.py
@@ -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 = [