ianByrne / HASS-ukho_tides

Home Assistant integration for tide information
MIT License
28 stars 6 forks source link

Additional Feature request - Tidal Height Offset? #20

Open Grey-Sheep opened 11 months ago

Grey-Sheep commented 11 months ago

Thanks for a great HA integration - this works so well 👍👍👍

It is good to see that it has an offset for High and Low tide times. For many installations (including my own 😁) a tidal height offset would also be very useful.

The sailing club is 15 minutes behind Osea Island times, but also 2.1M higher (so the tide has 2.1M less height)

The height may more complex than it first seems, as it will need to allow for areas that dry out, and so stay at zero for a period of time, so there is effectively no "Low Tide Time" which results in any graph having a flat bottom for part of that tidal cycle.

Many Thanks

ianByrne commented 11 months ago

Hi @Grey-Sheep , many thanks for your comment and for the suggestion. That does indeed sound like a useful feature request - I can certainly picture that scenario, in which "low tide" consists of 0m height for an extended period of time. I agree that it would require a bit of thought on how to implement, but I don't think that it should be too complex.

Unfortunately I do not have a lot of spare time lately, so will be unlikely to offer much help. However, I am happy to keep this issue open in the hopes that someone else might submit a PR.