search

radical-cybertools / radical.pilot

RADICAL-Pilot
http://radical-cybertools.github.io/radical-pilot/index.html
Other
54 stars 23 forks source link

`task.stdout` should be string, not a list #2746

Open andre-merzky opened 2 years ago

andre-merzky commented 2 years ago

we can expose task.stdout_per_rank additionally

eirrgang commented 2 years ago

In RP 1.18, Tasks (raptor tasks, at least) with multiple MPI ranks return a list for stdout. According to this issue, the type of the stdout field will be normalized to str in a future release.

So instead of analyzing the task to infer the field type, it is probably best for clients to account for a type of Union[str, list[str]] until we are able to assert that it is str, for some future RP version. (Future behavior will be to concatenate the various stdouts together, right? So clients can preemptively ''.join(stdout), I suppose.)