-
Notifications
You must be signed in to change notification settings - Fork 8
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
Import the LatestValueCache
implementation from the Frequenz SDK
#302
Conversation
f754374
to
9ffb428
Compare
Signed-off-by: Sahas Subramanian <sahas.subramanian@proton.me>
Signed-off-by: Sahas Subramanian <sahas.subramanian@proton.me>
Signed-off-by: Sahas Subramanian <sahas.subramanian@proton.me>
9ffb428
to
e5f661e
Compare
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.
LGTM, but I would try to keep the user guide documentation up to date too, can you maybe add some high-level documentation with some example to the new private module docstring and expose it via docs/user-guide/receiving/caching.md
maybe? Similar to this.
8ecdd87
to
897f2c6
Compare
Signed-off-by: Sahas Subramanian <sahas.subramanian@proton.me>
897f2c6
to
3e68eea
Compare
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.
Nice, thanks! I think I could make sense to rename get()
to value
and make it a @property
, but this can be discussed separately and done in a separate PR too, so approving anyways.
|
This PR also adds an integration test for
LatestValueCache
.