NA's in future_OM_list object cause obscure errors
Open
@k-doering-NOAA is already working on this.
Since Mar 18, 2022.
documentation
- Dominant language
- R
- Stars
- 21
- Forks
- 14
- PR merge metrics
- No merged PRs in 30d
Description
Need to confirm that the manual/example documentation makes it clear that all NA's in the future_OM_list object need to be replaced with real values or removed by making them NULL. Lots of NA's are created by the create_future_om_list() function so it is easy for users to leave them in by mistake. It may also be possible to add some code in SSMSE to automatically remove these values, I'm not sure how difficult that would be though.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.