Consider using .format() in string compositions

[GregVernon]

I've found that Cubit commands are often a little easier to read / more maintainable using the .format() method.

For instance, turn this:

paramak/paramak/parametric_neutronics/make_faceteted_neutronics_model.py

Lines 79 to 83 in 7924ab0

	cubit.cmd(
	"unite vol " +
	" ".join(new_vols) +
	" with vol " +
	" ".join(new_vols))

into:

cubit.cmd("unite vol {} with vol {}".format(new_vols, new_vols))

or a three-line statement (still fewer lines than the original):

command = "unite vol {} with vol {}"
command.format(new_vols, new_vols)
cubit.cmd(command)

[shimwell]

Thanks Greg this is a nice suggestion, thanks for taking the time to help out. We certainly appreciate it.

I has make a PR for this improvement, it will be a bit slower to merge than our usual PRs as to test this we need to connect to our Trelis license server and perform local tests.

I'm keen to update this script from its python 2 origins and also keen to run it using the system python instead of that os.system call that I current use which is a bit of a mess

paramak/paramak/parametric_neutronics/neutronics_model.py

Lines 366 to 367 in 8188fb5

	os.system("trelis -batch -nographics make_faceteted_neutronics_model.py \"faceting_tolerance='" +
	str(self.faceting_tolerance) + "'\" \"merge_tolerance='" + str(self.merge_tolerance) + "'\"")