Conversation
✅ Deploy Preview for mozilla-perfcompare ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
@mgaudet To resolve, I suggest using acronyms so MD(%) and write the full name in the tooltips. We'd also have to use CLES(%) for Effect Size. No need to add more info in the tooltip since it's already there.
Also, can you please run |
gmierz
left a comment
There was a problem hiding this comment.
Can you put this in the expanded row instead for each of the rows there (mean/median/variance/etc.)? We'd like to move away from focusing on the delta/magnitude of the change and having it at the row header goes against that. I can see it being useful though and having it in the expanded row would solve that if you ever need the number.
|
Not sure how to deal with stacked diffs here; but i have addressed feedback and have updates but on top of #1018 instead |
|
(As per discussion in Perfcompare channel, I've kept this as column as I see it as a hard requirement) I'm still struggling with box overflow a bit. I've dropped the text to MD%, but if you play with the deploy you'll see I put up #1018 to remove a column to make more room, to avoid blocking this, that can wait. |
|
Oh and apaprently npm test all and what is running on circleCI disagree... |




Allow sorting by magnitude for Mann Whitney U