-
Notifications
You must be signed in to change notification settings - Fork 90
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
BenchPlugin #618
BenchPlugin #618
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #618 +/- ##
============================================
- Coverage 48.00% 47.93% -0.07%
- Complexity 6180 6182 +2
============================================
Files 394 395 +1
Lines 38176 38235 +59
Branches 6085 6087 +2
============================================
+ Hits 18325 18327 +2
- Misses 18703 18762 +59
+ Partials 1148 1146 -2
☔ View full report in Codecov by Sentry. |
Pull Request ReviewHey there! 👋 Here's a summary of the previous tasks and their results. Let's dive in! Changes
Suggestions
BugsThere are no potential bugs found in the code. Improvements
private void addBenchPlugin(XYChart chart) {
chart.getPlugins().add(new BenchPlugin());
}
private void addBenchPlugin(XYChart chart) {
chart.getPlugins().add(new BenchPlugin());
} RatingOverall rating: 7.5/10
That's it for the summary! If you have any questions or need further clarification, feel free to reach out. Happy coding! 😄🚀 |
903bb72
to
b1b3731
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.
Looks good!
Adds an easy way for library users to use the basic features of the instrumentation and serves as a nice showcase in the samples.
6dce285
to
c9471cf
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Added an experimental
BenchPlugin
that displays a live benchmark of the current chart:Clicking again or closing the window disables the measurements again