From: kkangmin <kangminh925@gmail.com>
Date: Wed, 6 Aug 2025 07:42:38 +0000 (+0000)
Subject: Fix inconsistent indentation, typos, and missing parentheses
X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=commitdiff_plain;h=2cbc3bc82c4c8ef7ff326c34bb0dfe344b88d5be;p=aiml-fw%2Fawmf%2Ftm.git

Fix inconsistent indentation, typos, and missing parentheses

- In __getLeafPaths(): fixed missing parentheses, double quotes, and indentation
- In preprocessTrainingConfig(): fixed typo in function name
- In exception message: corrected 'recieved' to 'received'

Issue-Id: AIMLFW-223
Change-Id: Idf041c38d08139c1b0ab000a472403a823be7fa4
Signed-off-by: kkangmin <kangminh925@gmail.com>
---

diff --git a/trainingmgr/common/trainingConfig_parser.py b/trainingmgr/common/trainingConfig_parser.py
index 903419e..7e40c58 100644
--- a/trainingmgr/common/trainingConfig_parser.py
+++ b/trainingmgr/common/trainingConfig_parser.py
@@ -32,7 +32,7 @@ def parse_dict_by_fields(data, fields):
             cur = cur[field]
         return cur
     except Exception as e:
-        raise Exception("Can't parse Fields: {} in Data : {}| recieved-error : {}".format(fields, data, e))
+        raise Exception("Can't parse Fields: {} in Data : {}| received-error : {}".format(fields, data, e))
         
         
 def __getLeafPaths():
@@ -47,10 +47,11 @@ def __getLeafPaths():
                 "arguments": "{'epochs': '1'}"
             },
             "trainingPipeline": {
-                    "pipeline_name": "qoe_Pipeline",
-                    "pipeline_version": "2", 
-                    "retraining_pipeline_name":"retrain-qoe-pipeline",
-                    "retraining_pipeline_version:"2"
+                "pipeline_name": "qoe_Pipeline",
+                "pipeline_version": "2", 
+                "retraining_pipeline_name": "retrain-qoe-pipeline",
+                "retraining_pipeline_version": "2"
+            }
         }
     '''
     paths = {
@@ -61,11 +62,11 @@ def __getLeafPaths():
         "training_pipeline_name": ["trainingPipeline", "training_pipeline_name"],
         "training_pipeline_version": ["trainingPipeline", "training_pipeline_version"],
         "retraining_pipeline_name": ["trainingPipeline", "retraining_pipeline_name"],
-        "retraining_pipeline_version":["trainingPipeline", "retraining_pipeline_version"]
+        "retraining_pipeline_version": ["trainingPipeline", "retraining_pipeline_version"]
     }
     return paths
 
-def prepocessTrainingConfig(trainingConfig):
+def preprocessTrainingConfig(trainingConfig):
     if (isinstance(trainingConfig, str)):
         return json.loads(trainingConfig)
     return trainingConfig
@@ -75,7 +76,7 @@ def validateTrainingConfig(trainingConfig):
         One way to Validate TrainingConfig is to see if each Leafpath exists or not
         Any other key:value pair than TrainingConfig Schema is not treated as invalid. 
     '''
-    trainingConfig = prepocessTrainingConfig(trainingConfig)
+    trainingConfig = preprocessTrainingConfig(trainingConfig)
     allPaths = __getLeafPaths()
     try:
         for fieldPath in allPaths.values():
@@ -86,7 +87,7 @@ def validateTrainingConfig(trainingConfig):
         return False
     
 def getField(trainingConfig, fieldname):
-    trainingConfig = prepocessTrainingConfig(trainingConfig)
+    trainingConfig = preprocessTrainingConfig(trainingConfig)
     fieldPath = __getLeafPaths()[fieldname]
     return parse_dict_by_fields(trainingConfig, fieldPath)
 
@@ -94,7 +95,7 @@ def setField(trainingConfig, fieldname, value):
     """
     Set the value of a specific field in the training configuration.
     """
-    trainingConfig = prepocessTrainingConfig(trainingConfig)
+    trainingConfig = preprocessTrainingConfig(trainingConfig)
     fieldPath = __getLeafPaths()[fieldname]
     if not fieldPath:
         raise KeyError(f"Field '{fieldname}' is not defined in the schema.")