Skip to content
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

doc(how-to): Measure maximum latency #7

Merged
merged 24 commits into from
Mar 6, 2024

Conversation

locnnil
Copy link
Contributor

@locnnil locnnil commented Feb 26, 2024

  • add cyclictest examples
  • add histogram plot example

RTD preview

* add cyclictest examples
* add histogram plot example

Signed-off-by: Lincoln Wallace <[email protected]>
@lblythen lblythen self-requested a review February 26, 2024 21:45
@lblythen

This comment was marked as resolved.

@locnnil

This comment was marked as resolved.

@locnnil locnnil force-pushed the IENG-982-maximum-latency branch from fd496cd to 31b8f7a Compare February 27, 2024 01:14
@lblythen

This comment was marked as resolved.

Signed-off-by: Lincoln Wallace <[email protected]>
@locnnil

This comment was marked as resolved.

@lblythen lblythen marked this pull request as ready for review February 27, 2024 21:55
* add command output
* add instruction to install gnuplot
* add script to plot on doc to have more control on it

Signed-off-by: Lincoln Wallace <[email protected]>
locnnil and others added 4 commits February 27, 2024 22:21
locnnil and others added 5 commits February 28, 2024 15:28
Co-authored-by: Farshid Tavakolizadeh <[email protected]>
Co-authored-by: Farshid Tavakolizadeh <[email protected]>
Co-authored-by: Farshid Tavakolizadeh <[email protected]>
* Uses the same machine for all console outputs
* Add explanation on actions performed by script

Signed-off-by: Lincoln Wallace <[email protected]>
Signed-off-by: Lincoln Wallace <[email protected]>
@locnnil locnnil force-pushed the IENG-982-maximum-latency branch from 790a255 to 57383b0 Compare February 29, 2024 01:14
* use duration on histogram script
* add tab to show difference on outputs with and without rt
* add note with command on importance of loaded system when testing

Signed-off-by: Lincoln Wallace <[email protected]>
@locnnil locnnil requested review from farshidtz and EdoardoBarbieriCanonical and removed request for josephtsalisbury February 29, 2024 22:50
locnnil and others added 4 commits March 1, 2024 11:33
Co-authored-by: Farshid Tavakolizadeh <[email protected]>
Co-authored-by: Farshid Tavakolizadeh <[email protected]>
Co-authored-by: Farshid Tavakolizadeh <[email protected]>
* use only one machine
* add tabs to example results

Signed-off-by: Lincoln Wallace <[email protected]>
@locnnil locnnil force-pushed the IENG-982-maximum-latency branch from 364e803 to 74d311f Compare March 5, 2024 00:34
@locnnil locnnil requested a review from farshidtz March 5, 2024 00:45
Copy link
Member

@farshidtz farshidtz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. I appreciate the changes.
There are just a couple of minor issues related to the script and its citation.

locnnil and others added 4 commits March 5, 2024 10:21
Co-authored-by: Farshid Tavakolizadeh <[email protected]>
Co-authored-by: Farshid Tavakolizadeh <[email protected]>
@farshidtz farshidtz changed the title doc(how-to): Initial draft: measure maximum latency doc(how-to): Measure maximum latency Mar 6, 2024
@farshidtz farshidtz merged commit bb0b676 into canonical:main Mar 6, 2024
2 checks passed
@locnnil locnnil deleted the IENG-982-maximum-latency branch March 11, 2024 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants