Skip to content

Commit

Permalink
bug fix interactive mode
Browse files Browse the repository at this point in the history
  • Loading branch information
MichaelArbel committed Nov 20, 2023
1 parent 19e80ed commit e8a1310
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion mlxp/launcher.py
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,8 @@ def _main_job_command(executable, current_file_path, work_dir, parent_log_dir, j
+mlxp.logger.forced_log_id={job_id}\
+mlxp.logger.parent_log_dir={parent_log_dir} \
+mlxp.use_scheduler={False}\
+mlxp.use_version_manager={False}",
+mlxp.use_version_manager={False}\
+mlxp.interactive_mode={False}",
]

values = [f"{val}\n" for val in values]
Expand All @@ -431,6 +432,7 @@ def filter_fn(x):
and ("scheduler" not in x)
and ("logger.parent_log_dir" not in x)
and ("logger.forced_log_id" not in x)
and ("interactive_mode" not in x)
)

filtered_args = list(filter(filter_fn, overrides))
Expand Down

0 comments on commit e8a1310

Please sign in to comment.