search

MichaelFolkes / forecastR_package

forecastR package without shiny
1 stars 3 forks source link

Mechanistic return rate model "pred.label" argument #13

Open SOLV-Code opened 3 years ago

SOLV-Code commented 3 years ago

Current Setup

If the data frame fed into the mechanistic estimation fn has several columns starting with Pred_, and pred.label is not explicitly given as an input, it will take the first one from the left.

I included this for easier set-up in the app, where we can control the data input, and make sure it will always include just 1 pred_ column

Potential Issue

However, this creates potentially hidden functionality for package users, who might not be aware of that internal selection, and assume some kind of multi-variate step underneath the hood.

Team Decision

TBD

SOLV-Code commented 3 years ago

Executive decision: put it in the package documentation

SOLV-Code commented 3 years ago

note added to code script, still need to put into wiki and report