You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed two minor things in the documentation of the Stations.inventory method, which could need correction / clarification:
The documentation states that the argument required may be a list. However the type hint in the code does not include list and the code also checks for isinstance(required, tuple) and not for list. This could be corrected.
And is there any reason why granularity="monthly" is not allowed/listed? Currently only the granularity 'hourly' or 'daily' are listed in the documentation.
The text was updated successfully, but these errors were encountered:
I noticed two minor things in the documentation of the
Stations.inventory
method, which could need correction / clarification:required
may be alist
. However the type hint in the code does not includelist
and the code also checks forisinstance(required, tuple)
and not forlist
. This could be corrected.granularity="monthly"
is not allowed/listed? Currently only thegranularity
'hourly' or 'daily' are listed in the documentation.The text was updated successfully, but these errors were encountered: