search

CommonGarden / Grow-IoT

Software packages for smart growing environments.
http://commongarden.org/
Other
25 stars 6 forks source link

Not all emitted events are saved to local database. #446

Closed JakeHartnell closed 6 years ago

JakeHartnell commented 6 years ago

Currently, we use nedb for a lightweight local database used for storing event data. Unfortunately, not all events are being stored... I hope to figure this out soon so will either have a fix or steps to reproduce.

catbarph commented 6 years ago

Could we, by default, first store things like sensor readings to the local device, and only send data to cloud on significant change? Perhaps this could help reduce cloud database size?

ADD12 commented 6 years ago

I need this as well since many places we will want to use do not have a steady internet connection. Want to record offline and make AI adjustments locally in our Area Fog network. Cloud is ok to sync but steady connections are not possible. I have a mesh IIOT FOg network currently and puting 4 Intel Movidious Neural Compute sticks together for our other sensors. also want to add a Modem for remote connections to local fog network for access and control.

Alan DeRossett

On 2018-05-26 12:26, James Hauk wrote:

Could we, by default, first store things like sensor readings to the local device, and only send data to cloud on significant change? Perhaps this could help reduce cloud database size?

-- You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub [1], or mute the thread [2].

Links:

[1] https://github.com/CommonGarden/Grow-IoT/issues/446#issuecomment-392275835 [2] https://github.com/notifications/unsubscribe-auth/AHWdJmY-AgAIs7_3JtAYZqaDSpVz2Al5ks5t2ZBbgaJpZM4R6QDQ

JakeHartnell commented 6 years ago

I have a hunch moving the storage code (nedb or hypercore) to Thing.js will solve this. Things are a bit harder to keep track of in Grow.js, where we are dealing with an extension of an extension of an extension of the emit prototype. Arguably, this functionality belongs in Thing.js anyway. Will try to find time to do this soon.

JakeHartnell commented 6 years ago

Fixed in Thing.js v0.4.5

JakeHartnell commented 6 years ago

https://www.npmjs.com/package/Thing.js