searvey icon indicating copy to clipboard operation
searvey copied to clipboard

Historical NDBC stations

Open SorooshMani-NOAA opened this issue 1 year ago • 5 comments

In #146 historical stations are not yet implemented. See:

https://github.com/CDJellen/ndbc-api/blob/a4786307869d753e07c3413f1e1c0cf47f418ea2/ndbc_api/ndbc_api.py#L256

SorooshMani-NOAA avatar Sep 12 '24 20:09 SorooshMani-NOAA

I had a look on the faq: https://www.ndbc.noaa.gov/faq/measdes.shtml:

  • Realtime (the current implementation) is limited to the last 45days.
  • Historical is older than 45days

It seems we can get both data with get_data() function in NDBCApi.

tomsail avatar Sep 13 '24 07:09 tomsail

Yes, but the station list that we get right now is only from the active stations, not the historical ones (stations vs historical_stations).

SorooshMani-NOAA avatar Sep 13 '24 13:09 SorooshMani-NOAA

Some of these capabilities are very recently added to the upstream NDBC_API code, so it requires the latest version of API to work. So maybe it makes sense to go ahead with #146 first and later address this ticket. Previously (since 2 weeks ago?) there was a single stations query that addressed both!

SorooshMani-NOAA avatar Sep 13 '24 15:09 SorooshMani-NOAA

ok indeed. I think then it'd make sense to pin/limit the version of NDBCApi and merge #146

tomsail avatar Sep 13 '24 15:09 tomsail

Yes, it's already pinned

SorooshMani-NOAA avatar Sep 13 '24 16:09 SorooshMani-NOAA