Merge pull request #698 from linsword13/fix-warn

douglasjacobsen · web-flow · commit 355c24547d07 · 2024-10-15T19:57:48.000-06:00
Fix up the missing mpi_command warning
diff --git a/lib/ramble/ramble/application.py b/lib/ramble/ramble/application.py
@@ -1051,7 +1051,7 @@ def _define_commands(
                             and int(self.expander.expand_var_name(self.keywords.n_nodes)) > 1
                         ):
                             logger.warn(
-                                f"Command {cmd_conf} requires a non-empty `mpi_command` "
+                                f"Command {cmd_conf.name} requires a non-empty `mpi_command` "
                                 "variable in a multi-node experiment"
                             )
                         mpi_cmd = " " + raw_mpi_cmd + " "

Original file line number	Diff line number	Diff line change
`@@ -1051,7 +1051,7 @@ def _define_commands(`
`1051`	`1051`	`and int(self.expander.expand_var_name(self.keywords.n_nodes)) > 1`
`1052`	`1052`	`):`
`1053`	`1053`	`logger.warn(`
`1054`		- f"Command {cmd_conf} requires a non-empty `mpi_command` "
	`1054`	+ f"Command {cmd_conf.name} requires a non-empty `mpi_command` "
`1055`	`1055`	`"variable in a multi-node experiment"`
`1056`	`1056`	`)`
`1057`	`1057`	`mpi_cmd = " " + raw_mpi_cmd + " "`