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

Add a verbose argument to as_forecast() #780

Open
nikosbosse opened this issue Apr 5, 2024 · 2 comments
Open

Add a verbose argument to as_forecast() #780

nikosbosse opened this issue Apr 5, 2024 · 2 comments

Comments

@nikosbosse
Copy link
Contributor

Just we've done with validate_forecast() in #778 we should add a verbose argument for as_forecast()

@nikosbosse nikosbosse added this to the scoringutils-2.x milestone Apr 5, 2024
@seabbs
Copy link
Contributor

seabbs commented Apr 8, 2024

I don't think its really needed right now. I would vote for closing this and seeing if there is user demand (either from us or from others)

@nikosbosse
Copy link
Contributor Author

I think my personal user-demand would be to use this to get rid of a lot of print statements and suppressMessage() calls in the tests...
But yeah, definitely don't need to do that for the next CRAN release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants