search

ClemensElflein / OpenMower

Let's upgrade cheap off-the-shelf robotic mowers to modern, smart RTK GPS based lawn mowing robots!
Other
4.64k stars 274 forks source link

Stock-CoverUI integration proposal #65

Closed Apehaenger closed 1 year ago

Apehaenger commented 1 year ago

Hi Clemens!

This is only a proposal how I would/could integrated the Stock-UI packets into the LowLevel code. Not compiled nor tested! Simply written down to show how I would do it.

In this proposal I've chosen the way via two additional Stock-UI specific global vars, as by this I had the impression to get easiest integration into the existing code without disturbing logic or understanding.

What I'm unsure yet is: ~~- What get detected if no rain-sensor is connected to OM's mainboard? Same as with the halls? (not connected = no emergency/rain flag)~~

Apehaenger commented 1 year ago

Compiles well and work as expected ;-)

ClemensElflein commented 1 year ago

Thanks for the PR, looks good to me.

Apehaenger commented 1 year ago

That's cool, quite thanks!