From 0c3e16f46fbe616140e21985873fb9ddaf4793c9 Mon Sep 17 00:00:00 2001
From: Sushil Tiwari <susiltiwari750@gmail.com>
Date: Fri, 25 Oct 2024 10:59:46 +0545
Subject: [PATCH] Fix dtype issue and summary parsing issue

---
 per/ops_learning_summary.py | 13 ++++++++-----
 per/task.py                 |  2 +-
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/per/ops_learning_summary.py b/per/ops_learning_summary.py
index 197fc9258b..c28d103682 100644
--- a/per/ops_learning_summary.py
+++ b/per/ops_learning_summary.py
@@ -256,6 +256,7 @@ def fetch_ops_learnings(self, filter_data):
                 "region_name",
                 "appeal_name",
                 "appeal_year",
+                "dtype_name",
             )
         )
         ops_learning_df = ops_learning_df.rename(
@@ -436,7 +437,7 @@ def _identify_type_prioritization(df):
             return None
 
         def _contextualize_learnings(df):
-            """Adds appeal year and event name as a contextualization of the leannings."""
+            """Adds appeal year and event name as a contextualization of the learnings."""
             for index, row in df.iterrows():
                 df.at[index, "learning"] = (
                     f"{row['excerpts_id']}. In {row['appeal_year']} in {row['appeal_name']}: {row['learning']}"
@@ -738,8 +739,11 @@ def _validate_format(summary, MAX_RETRIES=3):
             """
 
             def _validate_text_is_dictionary(text):
-                formatted_text = ast.literal_eval(text)
-                return isinstance(formatted_text, dict)
+                try:
+                    formatted_text = ast.literal_eval(text)
+                    return isinstance(formatted_text, dict)
+                except Exception:
+                    return False
 
             def _modify_format(summary) -> str:
                 try:
@@ -752,8 +756,7 @@ def _modify_format(summary) -> str:
                     logger.info("Modification realized to response")
                     return formatted_summary
 
-                except Exception as e:
-                    logger.error(f"Modification failed: {e}", exc_info=True)
+                except Exception:
                     return "{}"
 
             formatted_summary = {}
diff --git a/per/task.py b/per/task.py
index dc4051ec99..9c03489cac 100644
--- a/per/task.py
+++ b/per/task.py
@@ -68,7 +68,7 @@ def generate_ops_learning_summary(ops_learning_summary_id: int, filter_data: dic
         OpsLearningSummaryTask.change_ops_learning_status(
             instance=ops_learning_summary_instance, status=OpsLearningCacheResponse.Status.NO_EXTRACT_AVAILABLE
         )
-        logger.error("No extracts found", exc_info=True)
+        logger.info("No extracts found")
         return False