Bug 2036968: Replaced fast-kde with fftkde and used bootstrap-ci to get CI summary #1034
Bug 2036968: Replaced fast-kde with fftkde and used bootstrap-ci to get CI summary #1034kala-moz wants to merge 4 commits intomozilla:mainfrom
Conversation
✅ Deploy Preview for mozilla-perfcompare ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1034 +/- ##
===========================================
- Coverage 96.32% 86.00% -10.32%
===========================================
Files 113 116 +3
Lines 3262 3695 +433
Branches 739 829 +90
===========================================
+ Hits 3142 3178 +36
- Misses 118 465 +347
- Partials 2 52 +50 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Good start! As discussed, let's switch to echarts for feature, performance and accessibility reasons. I've updated the examples in the other repo so that it is even more drop in, and added a "gallery" with multiple examples. The closest this PR looks like the examples the best it is. In particular, it's going to be a lot less visually noisy, with lots of useless numbers removed. This is a complaint I hear from perf people. lmk if we should do a quick 1:1 one off or pair to get this done! |
Thanks, @padenot. I went ahead and switched to echarts. Should I continue with the rest of the updates or should we break them into separate PRs? I feel this PR is pretty full already. |
This PR (Bug 2036968) integrates the js version of @padenot's standalone-stats into perfcompare. Here are the main changes for the integration:
Deploy Link
1.) Added
standalone-stats/kde.jsandkde.d.ts;bootstrap-ci.jsandbootstrap-ci.tstosrc/utils2.) In
src/components/CompareResults/CommonGraph.tsx- Removed the thefast-kdeimport and replaced with thefftkde. I removedapproximateSJBandwidthbecause the ISJ method below auto-selects the bandwidth per dataset. It also has an internal grid and quantile logic so didn't need thequantileSortedmethod.3.) Use the
bootstrapMedianDiffCIfrombootstrap-cito get the confidence interval summary and confidence interval statement.Note: please check that the comments are relevant as I may have missed to update or add them where necessary.
Still need to add coverage.
Updates: