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

docs: Add summary of the current performance measurement #37

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

ueno
Copy link
Contributor

@ueno ueno commented Sep 11, 2023

This adds a document that summarizes our experiment to measure performance impact of applications being audited.

Copy link
Contributor

@neverpanic neverpanic left a comment

Choose a reason for hiding this comment

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

Minor issues only, looks good to me otherwise.

docs/performance.md Outdated Show resolved Hide resolved
docs/performance.md Outdated Show resolved Hide resolved
docs/performance.md Outdated Show resolved Hide resolved
docs/performance.md Outdated Show resolved Hide resolved
Copy link

@ZoltanFridrich ZoltanFridrich left a comment

Choose a reason for hiding this comment

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

Overall looks good. I just have a few suggestions to reword some sentences.

docs/performance.md Outdated Show resolved Hide resolved
docs/performance.md Outdated Show resolved Hide resolved
docs/performance.md Outdated Show resolved Hide resolved
docs/performance.md Outdated Show resolved Hide resolved
This adds a document that summarizes our experiment to measure
performance impact of applications being audited.

Signed-off-by: Daiki Ueno <[email protected]>
Copy link

@ZoltanFridrich ZoltanFridrich left a comment

Choose a reason for hiding this comment

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

LGTM!

@ueno
Copy link
Contributor Author

ueno commented Sep 12, 2023

Thank you for the reviews!

@ueno ueno merged commit a11d144 into latchset:main Sep 12, 2023
6 checks passed
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.

3 participants