Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Advanced performance metrics#790

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
sharkdp wants to merge40 commits intomaster
base:master
Choose a base branch
Loading
fromnew-metrics
Draft

Advanced performance metrics#790

sharkdp wants to merge40 commits intomasterfromnew-metrics

Conversation

@sharkdp
Copy link
Owner

@sharkdpsharkdp commentedDec 29, 2024
edited
Loading

New features

  • Better JSON export format (see below)
  • Properly support peak memory usage as a new metric
  • Support new time units: minutes and hours

To do

  • serialize unit information (long/short)
  • (ExitStatus, Second, Second, u64) =>Measurement?
  • Further flesh outMeasurement
  • Get rid ofTimeQuantity::to_string?
  • Get rid of all new TODOs in the code
  • Review occurrences of.value_in
  • Re-add aggregate statistics to JSON export?
  • Add command line invocation to JSON export?
  • Move outlier detection to separate module
  • Support alternative target metrics

JSON format

Current version of JSON format:
{"results": [    {"command":"fd -ug blob ~","measurements": [        {"time_wall_clock": {"value":0.19842726476,"unit":"second"          },"time_user": {"value":0.75561904,"unit":"second"          },"time_system": {"value":2.08008938,"unit":"second"          },"peak_memory_usage": {"value":118804480,"unit":"byte"          },"exit_code":0        },        {"time_wall_clock": {"value":0.19565810476,"unit":"second"          },"time_user": {"value":0.83431904,"unit":"second"          },"time_system": {"value":2.00121838,"unit":"second"          },"peak_memory_usage": {"value":117673984,"unit":"byte"          },"exit_code":0        }      ]    },    {"command":"find ~ -name blob","measurements": [        {"time_wall_clock": {"value":2.45530797276,"unit":"second"          },"time_user": {"value":0.63306604,"unit":"second"          },"time_system": {"value":1.81707638,"unit":"second"          },"peak_memory_usage": {"value":10592256,"unit":"byte"          },"exit_code":0        },        {"time_wall_clock": {"value":2.45145763076,"unit":"second"          },"time_user": {"value":0.62730604,"unit":"second"          },"time_system": {"value":1.81925538,"unit":"second"          },"peak_memory_usage": {"value":10686464,"unit":"byte"          },"exit_code":0        }      ]    }  ]}

konstin and mati865 reacted with heart emoji
@sharkdpsharkdpforce-pushed thenew-metrics branch 2 times, most recently fromcb9a136 toab811ddCompareDecember 29, 2024 15:29
@sharkdpsharkdp changed the titleNew metricsAdvanced performance metricsDec 29, 2024
@sharkdpsharkdpforce-pushed thenew-metrics branch 3 times, most recently from40dba4f to3cdae0fCompareJanuary 4, 2025 21:27
@krkkrk mentioned this pull requestMay 21, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sharkdp

[8]ページ先頭

©2009-2025 Movatter.jp