search

open-meteo / python-requests

Open-Meteo Python Library using `requests`
MIT License
15 stars 4 forks source link

merge openmeteopy into python-requests #34

Closed m0rp43us closed 5 months ago

m0rp43us commented 9 months ago

Hello,

I can add a third version based on the sdk schemas created & add more functionnalities.

patrick-zippenfenig commented 9 months ago

Dear @m0rp43us,

sorry for not proactively reaching out. This python client is only a very shim layer and does not offer much functionally except decoding the FlatBuffer binary format (barely 30 lines of code). I am still considering if this is the best way forward as this schema may complicate usage.

Could you have a look if the SDK schema with FlatBuffers makes sense for openmeteopy? Depending on the design, it may not require a new major version bump

m0rp43us commented 9 months ago

Dear @patrick-zippenfenig , I took a look on the sdk and it seems to me that there is a way to integrate FlatBuffers. I will release a first stable version of the Python library (with tests and a Pypi release) at the beginning of 2024. I do like the FlatBuffers optimization choice and also the models,makes everything much more simple regarding sdks.