ajsims1704 / rdecision

Decision Analytic Modelling In Health Economics
1 stars 0 forks source link

Tornado function requires arguments ref and index to be lists, even if they are single elements. #17

Open ajsims1704 opened 3 months ago

ajsims1704 commented 3 months ago

The 'index' and 'ref' arguments of DecisionTree$tornado expect strategies to be given as lists of action edges. In the case of a strategy with only one action edge (as in, for example, a simple two choice decision tree), it seems redundant to have a wrap the action edge in a list(). Consider whether these arguments could accept single action edges as well as lists of edges.

Related, the description of these arguments should be clarified in the documentation to avoid confusion.