-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fix typo in pmem calculation AND disable Mac Tests #151
Conversation
Fixes con#150 Also - use floats rather than ints for 0 - add % to human readable summary
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #151 +/- ##
==========================================
- Coverage 96.36% 94.67% -1.69%
==========================================
Files 2 2
Lines 440 451 +11
Branches 66 69 +3
==========================================
+ Hits 424 427 +3
- Misses 9 13 +4
- Partials 7 11 +4 ☔ View full report in Codecov by Sentry. |
=== Do not change lines below === { "chain": [], "cmd": "./.update-readme-help.py", "exit": 0, "extra_inputs": [], "inputs": [], "outputs": [], "pwd": "." } ^^^ Do not change lines above ^^^
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.
add a test .
could mimic original invocation which observed that odd behavior: run duct duct sleep 1
with some short periods for sampling/report
Ensure, for example, that percentages are not large values
tests aren't happy , hadn't looked in detail yet. |
All the MacOS tests are failing because we cannot collect samples. The SANITY commit was dropped but it gave plenty of time to collect a sample, but no samples were collected. This is related to #82 |
The tests didnt run, looks like a partial outage |
Co-authored-by: Yaroslav Halchenko <[email protected]>
Fixes #150
Also