-
Notifications
You must be signed in to change notification settings - Fork 301
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
SR860: Add snapshot_base to prevent update of parameter that may not be defined #1013
SR860: Add snapshot_base to prevent update of parameter that may not be defined #1013
Conversation
params_to_skip_update = [] | ||
# we omit count_capture_kilobytes from the snapshot because | ||
# it can only be read after a completed capture and will | ||
# timeout otherwise when the snapshot is updated, I.e at |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i.e. -> e.g. (I would say)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left one pesky comment, else super 👍
Codecov Report
@@ Coverage Diff @@
## master #1013 +/- ##
=========================================
Coverage ? 78.96%
=========================================
Files ? 45
Lines ? 6445
Branches ? 0
=========================================
Hits ? 5089
Misses ? 1356
Partials ? 0 |
Author: Jens Hedegaard Nielsen <jenshnielsen@gmail.com> SR860: Add snapshot_base to prevent update of parameter that may not be defined (#1013)
This parameter only works after a complete capture so any automatic update will most likely fail. This cuts 5 sec off the time needed to add this to the station