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

better error message for bad article ID? #22

Closed
bbolker opened this issue Nov 7, 2015 · 2 comments
Closed

better error message for bad article ID? #22

bbolker opened this issue Nov 7, 2015 · 2 comments

Comments

@bbolker
Copy link

bbolker commented Nov 7, 2015

This StackOverflow question clearly reflects user error/confusion, but the error message that you get when putting in a bogus article ID,

Error in min(years):max(years) : result would be too long a vector
In addition: Warning messages:
1: In min(years) : no non-missing arguments to min; returning Inf
2: In max(years) : no non-missing arguments to max; returning -Inf

is not terribly transparent to new users, and could probably be clearer ...

@jkeirstead
Copy link
Owner

Thanks for raising this Ben. I'll add some checking of the arguments so that a more sensible error is thrown before it tries to create an infinitely long sequence of years!

@jefferis
Copy link
Collaborator

There is a fix for this in develop.

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

3 participants