Decrease wait time for commands about farmbot_os HOT 6 CLOSED

The idea there is that the system waits for a refresh signal during one
minute (it's named 'refresh' because I use this method a lot to trigger
screen refreshes when new data arrives). If after one minute there is no
signal received, it will check the database to see if a command has been
received without setting the refresh flag. So that signal goes wrong. That
would explain it.

On Tue, Feb 24, 2015 at 4:11 AM, Rory Aronson [email protected]
wrote:

Hey @TimEvWw https://github.com/TimEvWw, Rick and I were messing with
the latest bot and found that commands had a delay of about a minute. We
looked into the controller source code and found this:

https://github.com/FarmBot/farmbot-raspberry-pi-controller/blob/bbc816cb80ad84150404a26872ff75660da93c42/lib/controller.rb#L128

Is there a reason the + 60 is there?

—
Reply to this email directly or view it on GitHub
#50.

from farmbot_os.

Do you think there will be any repercussions for lowering the number more, maybe to 1?

from farmbot_os.

Higher system load mainly. SQLite was a bit sluggish for continuous polling
for my taste (hence the refresh system) but it works as a quick fix.

On Tue, Feb 24, 2015 at 3:07 PM, Rick Carlino [email protected]
wrote:

Do you think there will be any repercussions for lowering the number
lower, like to 1?

—
Reply to this email directly or view it on GitHub
#50 (comment)
.

from farmbot_os.

Ok, so menu.rb is scheduling a command, but not setting the refresh flag?

I removed the +60 altogether locally and watched the CPU go to 85% haha

from farmbot_os.

I think that the setting of the flag was done in the same routine as writing of the command but it was lost in refactoring. Will check.

Op 24-feb.-2015 om 22:35 heeft Rory Aronson [email protected] het volgende geschreven:

Ok, so menu.rb is scheduling a command, but not setting the refresh flag?

I removed the +60 altogether locally and watched the CPU go to 85% haha

—
Reply to this email directly or view it on GitHub.

from farmbot_os.

Refresh is modified.

from farmbot_os.