You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I now see why I was somewhat confused on the original PR. The name for the value returned from the workflow is rx_value but the value returned is the estimated sex from the genome sequence data. One of the values in the TSV is the actual Rx value (a numeric, decimal value between 0-1). As discussed, the actual Rx value is ok to omit as a workflow output.
Instead of naming the workflow output as rx_value perhaps it should be named to rx_sex_estimate to avoid further confusion?
The text was updated successfully, but these errors were encountered:
qc-analysis-pipeline/PreindexedRxIdentifier.wdl
Line 57 in ff7f603
I now see why I was somewhat confused on the original PR. The name for the value returned from the workflow is
rx_value
but the value returned is the estimated sex from the genome sequence data. One of the values in the TSV is the actual Rx value (a numeric, decimal value between 0-1). As discussed, the actual Rx value is ok to omit as a workflow output.Instead of naming the workflow output as
rx_value
perhaps it should be named torx_sex_estimate
to avoid further confusion?The text was updated successfully, but these errors were encountered: