Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an option to ignore the MQTT payload datetime and process the queue until it is empty #25

Closed
bellrichm opened this issue Jun 25, 2019 · 2 comments

Comments

@bellrichm
Copy link
Owner

This is for the service, just need to make sure to not break the driver.
Also, if data comes in very frequently from MQTT, there is a chance that we could get stuck in a loop just pulling data from MQTT.

bellrichm added a commit that referenced this issue Jun 26, 2019
@bellrichm
Copy link
Owner Author

Current plan is to add two options to the [[topics]] section, ignore_packet_start_time and ignore_packet_end_time. Both of these will default to False.
When the ignore_packet_start_time is True, the MQTT data datetime will be not be checked that is greater than the last packet processed.
When the ignore_packet_end_time is True, the MQTT data will continue to be processed even if its datetime is greater than the packet’s datetime. But, if more data arrives as data is being pulled from MQTT, this data will be left for the next packet. This check ensures that we don’t get stuck in a loop pulling data from MQTT.

@bellrichm
Copy link
Owner Author

Decided on ignore_start_time and ignore_end_time.

bellrichm added a commit that referenced this issue Jul 12, 2019
* option to ignore datetime in MQTT payload

resolves #24

* update version

* initial pass at issue #25

* first pass recfactor overlap logic

* rename config setting and variables

* change option and variable name

* first pass recfactor overlap logic

* rename config setting and variables

* consistent naming

resolves #26

* Finis commenting

* update change log

* up version

* Update README.md

* Update README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant