I seem to randomly get a 'station' error when running my_client.get_data_by_station inside of a for loop.

[DominickAugustine]

[Jwely]

@DominickAugustine

This appears to have been introduced by a new bug in the API (an old bug by time I've come around to noticing it). The Error you experienced was from the attempted handling of results which were empty due to a <500> response produced by this bug.

It is triggered by use of the units argument. I have updated the code to allow the units argument to be omitted by passing the default_units=None keyword argument when instantiating your client. So...

my_client = Client(token, default_units=None, default_limit=1000)

This means the results of your query will take the API default units, which can be a little bit unexpected, such as tenths of degrees Celsius, but valid requests are returned.

As an example for the API maintainers, the example requests in the documentation here do not work with the units='metric' or units='standard' arguments. They work fine however if the arguments are omitted.