Allow multiple output file names to be collected into array.
alexmorley opened this issue · 5 comments
At the moment
--inputs input:file1 input:file2
is allowed but
--outputs output:file1 output:file2
isn't.
Ideally they would both be allowed as it can be useful to have an array of output file names collected automatically as well. @magland was this deliberate?
relevant functions are in mlproc/run_process.js:
check_inputs_and_substitute_prvs # already works
check_outputs_and_substitute_prvs # to update
make_temporary_outputs # to update
@alexmorley I just hadn't encountered a situation where this was needed. I agree, it would be good to have this capability. This change/enhancement would need to be propagated to the various components (ml-js, lari, mlclient, mountainlab_pytools, ...)
OK, I needed it so I have implemented it in mlproc (on my own fork of ml-js) but not got round to propagating through yet.
- mlproc
- lari
- mlclient
- mountainlab_pytools
Do u have a pull request for it?
Not yet. Soon!