From 0bc3cdba05820be99a7da1ecc87b79cc35194d00 Mon Sep 17 00:00:00 2001
From: justin-richling <richling@ucar.edu>
Date: Tue, 9 Jan 2024 12:11:47 -0700
Subject: [PATCH] Update amwg_table.py

Bring in changes from PR #240
---
 scripts/analysis/amwg_table.py | 53 +++++-----------------------------
 1 file changed, 8 insertions(+), 45 deletions(-)

diff --git a/scripts/analysis/amwg_table.py b/scripts/analysis/amwg_table.py
index e7765d3cb..97d9a2cbf 100644
--- a/scripts/analysis/amwg_table.py
+++ b/scripts/analysis/amwg_table.py
@@ -150,9 +150,6 @@ def amwg_table(adf):
         output_locs.append(output_locs[0])
 
     #-----------------------------------------
-    #Create (empty) dictionary to use for the
-    #residual top of model (RESTOM) radiation calculation:
-    restom_dict = {}
 
     #Loop over CAM cases:
     for case_idx, case_name in enumerate(case_names):
@@ -182,9 +179,6 @@ def amwg_table(adf):
             Path.unlink(output_csv_file)
         #End if
 
-        #Save case name as a new key in the RESTOM dictonary:
-        restom_dict[case_name] = {}
-
         #Create/reset new variable that potentially stores the re-gridded
         #ocean fraction xarray data-array:
         ocn_frc_da = None
@@ -270,14 +264,6 @@ def amwg_table(adf):
                 # Note: we should be able to handle (lat, lon) or (ncol,) cases, at least
                 data = pf.spatial_average(data)  # changes data "in place"
 
-            #Add necessary data for RESTOM calcs below
-            if var == "FLNT":
-                restom_dict[case_name][var] = data
-                #Copy units for RESTOM as well:
-                restom_units = unit_str
-            if var == "FSNT":
-                restom_dict[case_name][var] = data
-
             # In order to get correct statistics, average to annual or seasonal
             data = pf.annual_mean(data, whole_years=True, time_name='time')
 
@@ -305,35 +291,13 @@ def amwg_table(adf):
         #End of var_list loop
         #--------------------
 
-        if "FSNT" and "FLNT" in var_list:
-            #RESTOM Calcs
-            var = "RESTOM" #RESTOM = FSNT-FLNT
-            print(f"\t - Variable '{var}' being added to table")
-            data = restom_dict[case_name]["FSNT"] - restom_dict[case_name]["FLNT"]
-            # In order to get correct statistics, average to annual or seasonal
-            data = pf.annual_mean(data, whole_years=True, time_name='time')
-            # These get written to our output file:
-            stats_list = _get_row_vals(data)
-            row_values = [var, restom_units] + stats_list
-            # col (column) values declared above
-
-            # Format entries:
-            dfentries = {c:[row_values[i]] for i,c in enumerate(cols)}
-
-            # Add entries to Pandas structure:
-            df = pd.DataFrame(dfentries)
-
-            # Check if the output CSV file exists,
-            # if so, then append to it:
-            if output_csv_file.is_file():
-                df.to_csv(output_csv_file, mode='a', header=False, index=False)
-            else:
-                df.to_csv(output_csv_file, header=cols, index=False)
-            #End if
-
-        else:
-            #Print message to debug log:
-            adf.debug_log("RESTOM not calculated because FSNT and/or FLNT variables not in dataset")
+        # Move RESTOM to top of table
+        #----------------------------
+        if 'RESTOM' in var_list:
+            table_df = pd.read_csv(output_csv_file)
+            table_df = pd.concat([table_df[table_df['variable'] == 'RESTOM'], table_df]).reset_index(drop = True)
+            table_df = table_df.drop_duplicates()
+            table_df.to_csv(output_csv_file, header=cols, index=False)
         #End if
 
         # last step is to add table dataframe to website (if enabled):
@@ -343,7 +307,6 @@ def amwg_table(adf):
     #End of model case loop
     #----------------------
 
-
     #Notify user that script has ended:
     print("  ...AMWG variable table has been generated successfully.")
 
@@ -434,4 +397,4 @@ def _df_comp_table(adf, output_location, case_names):
     adf.add_website_data(df_comp, "Case Comparison", case_names[0], plot_type="Tables")
 
 ##############
-#END OF SCRIPT
+#END OF SCRIPT
\ No newline at end of file