Skip to content

Commit b08779b

Browse files
committed
Fix: MLflow nested run error and limit Random Forest parallelism in CI
1 parent c80509d commit b08779b

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

‎Scripts/model_training.py‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -469,6 +469,9 @@ def register_model_to_registry(self, model_name: str, model, metrics: ModelMetri
469469
registry_name = registry_name or model_name
470470

471471
try:
472+
# End any active MLflow run before starting a new one
473+
if self.mlflow.active_run() is not None:
474+
self.mlflow.end_run()
472475
# Start MLflow run
473476
with self.mlflow.start_run(run_name=f"{model_name}_registration") as run:
474477
# Log model parameters
@@ -1299,7 +1302,7 @@ def main():
12991302
'min_samples_split': lambda trial: trial.suggest_int('min_samples_split', 100, 200), # High split requirement
13001303
'min_samples_leaf': lambda trial: trial.suggest_int('min_samples_leaf', 50, 100), # Min 50 for robust leaves
13011304
'criterion': lambda trial: trial.suggest_categorical('criterion', ['gini', 'entropy']),
1302-
'n_jobs': lambda trial: -1, # Use all CPU cores
1305+
'n_jobs': (lambda trial: 1) if is_ci else (lambda trial: -1), # Limit parallelism in CI
13031306
'class_weight': lambda trial: 'balanced' # Handle class imbalance
13041307
}
13051308

0 commit comments

Comments
 (0)