-
Notifications
You must be signed in to change notification settings - Fork 4
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
Provide non blocking API #16
Conversation
b0b6342
to
87727af
Compare
Codecov Report
@@ Coverage Diff @@
## master #16 +/- ##
==========================================
+ Coverage 85.34% 86.47% +1.12%
==========================================
Files 3 3
Lines 116 133 +17
==========================================
+ Hits 99 115 +16
- Misses 17 18 +1
|
87727af
to
5ad8764
Compare
That way the user can choose to just start a measurement and read the results later.
5ad8764
to
7df48ff
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.
Can you provide multiple impl blocks (common functionality, blocking API, non-blocking API) so we get nicer docs?
Most of the time delays need to be used all over in the system, thus we can't just take ownership of it.
757e35a
to
046e47a
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.
Thanks! There are some more things that should be done before the next release:
- Provide an async wakeup function as well
- Add notes about async to README and to crate-level documentation
- Add an example that uses async calls (ideally an embedded one)
However all of them can be added in separate PRs, so let's merge this.
@rnestler can you squash the last two commits before merging?
That way the documentation is easier to read, since the async and blocking measurement functions are separate. Co-authored-by: Danilo Bargen <[email protected]>
7691f2d
to
5c4bdd7
Compare
No description provided.