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

why 20 station limit per query? #29

Open
kendonB opened this issue May 3, 2021 · 1 comment
Open

why 20 station limit per query? #29

kendonB opened this issue May 3, 2021 · 1 comment

Comments

@kendonB
Copy link

kendonB commented May 3, 2021

I tried the query with 21 and it seemed to work fine but perhaps I missed something.

thanks for a nice package!

@kendonB kendonB changed the title why 20 station limit to queries? why 20 station limit per query? May 3, 2021
@blasee
Copy link
Contributor

blasee commented May 21, 2021

Hi @kendonB,

If the number of stations is more than 20 then the code will run fine, but it will only query for the first 20 stations. You will get a message stating that the function is only "using the first 20 stations" to form the query. The only reason I chose to impose a 20 station limit was to be kind to the server and to discourage bots programmatically downloading large amounts of data from CliFlo at once.

Thanks for your feedback :-)

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

No branches or pull requests

2 participants