[BUG]: fix configuration file for tweedie() models or distributions with 1 model

pfmc-assessments / indexwc

Estimate indices of abundance for west coast fish species

2 stars 1 forks source link

[BUG]: fix configuration file for tweedie() models or distributions with 1 model #30

Open kellijohnson-NOAA opened 6 months ago

kellijohnson-NOAA commented 6 months ago

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

The spatial component for the second model needs to be NA for distributions like the tweedie with just one model whereas a delta model needs two configurations for the spatial component.

Expected Behavior

That things work.

Steps To Reproduce

No response

Environment

- OS:Windows
- Node:
- npm:

Anything else?

Should also check that the diagnostics actually work.

kellijohnson-NOAA commented 5 months ago

An example of what would work as an entry for the spatiotemporal column, which is a concatenation of the spatiotemporal1 and spatiotemporal2 columns

"list('iid','off')"
"c('iid')"

Everything needs to be a string, and if that string contains strings, then single quotes must be used inside of the double quotes, i.e., "iid" will not work. If anyone knows how to do quosures like how tidyverse style programming does them so we do not have to double quote things, I am all ears.