cohorts icon indicating copy to clipboard operation
cohorts copied to clipboard

Support NA PFS or OS

Open arahuja opened this issue 9 years ago • 2 comments

Maybe out of scope for this project, but for TCGA data it would be useful to support PFS or OS being set to NaN. Right now, this gives an error because of the PFS <= OS assertion

arahuja avatar May 27 '16 17:05 arahuja

I think that's fair game. cohorts seems to support several random use cases; e.g. it's even useful for both genetic data and purely clinical data. Eventually, instead of None in a million places, it might be nice to clearly delineate some categories of use (and eventually split them out). But None in a lot of places seems fine for now.

tavinathanson avatar May 27 '16 19:05 tavinathanson

This will actually be important for the RCC analysis, where we expect a subset of patients to have temporarily unknown survival status until time of unblinding

jburos avatar Oct 24 '16 16:10 jburos