TheJacksonLaboratory / LIRICAL

LIkelihood Ratio Interpretation of Clinical AbnormaLities
https://thejacksonlaboratory.github.io/LIRICAL/stable
Other
22 stars 11 forks source link

adding checks that -m and -t cannot be used together and that -t \in [0,1] #479

Closed pnrobinson closed 4 years ago

pnrobinson commented 4 years ago

Added some non-elegant but straightforward code to check this. There is no way to enforce mutual exclusivity with JCommander apparently, and it would require a rewrite of LiricalFactory to do this in a really elegant way. We might consider using Spring to de-spaghettify the construction of objects in LIRICAL, and so I hope we can leave this for now.