clean up

Author: TejasMorbagal

deep_code/utils/dataset_stac_generator.py

@@ -519,13 +519,13 @@ def build_dataset_stac_collection(self) -> Collection:
             collection.extra_fields["themes"] = [theme_obj]
 
             for theme in self.osc_themes:
-                formated_theme = self.format_string(theme)
+                formatted_theme = self.format_string(theme)
                 collection.add_link(
                     Link(
                         rel="related",
                         target=f"../../themes/{theme}/catalog.json",
                         media_type="application/json",
-                        title=f"Theme: {formated_theme}",
+                        title=f"Theme: {formatted_theme}",
                     )
                 )
 

deep_code/utils/github_automation.py

@@ -49,7 +49,6 @@ def clone_sync_repository(self):
         """Clone the forked repository locally if it doesn't exist, or pull updates if it does."""
         logging.info("Checking local repository...")
         if not os.path.exists(self.local_clone_dir):
-            # Directory doesn't exist, clone the repository
             logging.info("Cloning forked repository...")
             try:
                 subprocess.run(
@@ -60,7 +59,6 @@ def clone_sync_repository(self):
             except subprocess.CalledProcessError as e:
                 raise RuntimeError(f"Failed to clone repository: {e}")
         else:
-            # Directory exists, pull the latest changes
             logging.info("Local repository already exists. Pulling latest changes...")
             try:
                 os.chdir(self.local_clone_dir)
@@ -81,26 +79,20 @@ def create_branch(self, branch_name: str):
     def add_file(self, file_path: str, content):
         """Add a new file to the local repository."""
         logging.info(f"Adding new file: {file_path}...")
-        os.chdir(self.local_clone_dir)  # Ensure we are in the Git repository
+        os.chdir(self.local_clone_dir)
         full_path = Path(self.local_clone_dir) / file_path
         full_path.parent.mkdir(parents=True, exist_ok=True)
-
         # Ensure content is serializable
         if hasattr(content, "to_dict"):
             content = content.to_dict()
         if not isinstance(content, (dict, list, str, int, float, bool, type(None))):
             raise TypeError(f"Cannot serialize content of type {type(content)}")
-
-        # Serialize to JSON
         try:
             json_content = json.dumps(content, indent=2, default=serialize)
         except TypeError as e:
             raise RuntimeError(f"JSON serialization failed: {e}")
-
         with open(full_path, "w") as f:
             f.write(json_content)
-
-        # Git add the file
         try:
             subprocess.run(["git", "add", str(full_path)], check=True)
         except subprocess.CalledProcessError as e:

deep_code/utils/helper.py

@@ -8,7 +8,7 @@ def serialize(obj):
         TypeError: If the object cannot be serialized.
     """
     if isinstance(obj, set):
-        return list(obj)  # Convert sets to lists
+        return list(obj)
     if hasattr(obj, "__dict__"):
-        return obj.__dict__  # Convert objects with attributes to dicts
+        return obj.__dict__
     raise TypeError(f"Object of type {type(obj).__name__} is not JSON serializable")

deep_code/utils/ogc_record_generator.py

@@ -61,9 +61,7 @@ def build_record_properties(
 
         if themes_list:
             theme_obj = self.build_theme(themes_list)
-            properties.update(
-                {"themes": [theme_obj]}
-            )
+            properties.update({"themes": [theme_obj]})
 
         properties.setdefault("type", "workflow")
         properties.setdefault("osc_project", "deep-earth-system-data-lab")