There is a known bug in the climate.NOAA function that lead to an error where urllib sends back a HTTP Error 400. Investigating the cause for this error.
Hi @exTerEX, this is a cool project, but it looks like there's some leftover code in the main branch that breaks imports if you install from source, and might be exposing an API key.
Running import noaa after installing with python -m pip install git+https://github.com/exTerEX/noaa.git#egg=noaa throws:
Expand on the pipeline in .github/workflows/release.yml to automatically put wheel in the release document. A possible actions could be: actions/upload-release-asset.
The sphinx documentation needs expansion. Needs to include allowed values, etc. in documentation.rst, and examples of how to use the code. A minimum of one simple example in index.rst and potentially a few more intermediate to advanced uses in examples.rst.
Version: main branch head, but looks like it affects v0.1.0rc2 too
Platform: Windows 10
Subsystem:
The get_data method isn't working because it uses malformed date parameters. This happens within _call_api when _full_time==True because it appends a time stamp here and here onto the ISO-formatted datetime that already includes a timestamp.
For example, datetime.datetime.fromisoformat("2000-01-01").isoformat() returns '2000-01-01T00:00:00', so manually appending a timestamp results in an invalid datetime '2000-01-01T00:00:00T00:00:00' and causes the server to return a 400 error.
Just having get_data call _call_api with _full_time=False fixes the issue, but maybe that option can be removed entirely if there's no need for datetimes without a timestamp. This has been a really helpful package, so I'm happy to make a PR for this is you want.