Question: Is there a performance reason to do the map within the function instead of as an argument in the target?
Context for the question/alternate approach to think about: If branches are used in the target argument, we could maybe get any failed site info pulls on a re-run, such that you don't need the result/error step here. It feels a little more "aligned" with the rest of the workflow that uses branches quite frequently.
mbrousil
commented
6 months ago
Question: Is there a performance reason to do the map within the function instead of as an argument in the target?
Context for the question/alternate approach to think about: If branches are used in the target argument, we could maybe get any failed site info pulls on a re-run, such that you don't need the result/error step here. It feels a little more "aligned" with the rest of the workflow that uses branches quite frequently.
_Originally posted by @steeleb in https://github.com/rossyndicate/AquaMatch_harmonize_WQP/pull/86#discussion_r1557902890_
See if
map()will work in thetar_target()part of this