Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use checkmate for argument checking #45

Closed
athowes opened this issue May 17, 2024 · 2 comments
Closed

Use checkmate for argument checking #45

athowes opened this issue May 17, 2024 · 2 comments
Assignees

Comments

@athowes
Copy link
Collaborator

athowes commented May 17, 2024

Have lots of functions which use something like a data.table with certain column names. Need to make sure the inputs to these functions are valid. Currently not doing this. See: https://mllg.github.io/checkmate/

@seabbs
Copy link
Contributor

seabbs commented May 18, 2024

We also have a specific data.table checker in Epinowcast that @pearsonca is spinning out into coerce_dt. Not sure we need that here though.

@athowes
Copy link
Collaborator Author

athowes commented Jul 11, 2024

Going to close this issue as I have now used checkmate in closed PRs and would be confident about using it in future work.

@athowes athowes closed this as completed Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants