catchpoint / WebPageTest

Official repository for WebPageTest
Other
3.08k stars 713 forks source link

Last Painted Hero (LPH) metric is overly sensitive to browser artifacts #1274

Closed theball closed 5 months ago

theball commented 5 years ago

See the "Waiting for..." text at the bottom of this image, which comes from the browser, not the web page.

image

Those updates increase the LPH number. See the full filmstrip here. Perhaps a simple fix is to ignore the bottommost pixels of the viewport.

I'm excited about LPH. It seems like the metric that most closely aligns to user perceived page load. Speed Index is good but IMO gives too much credit to the pattern of showing shimmers/skeletons so I'd love to see some of these kinks with LPH resolved.

/cc @paulirish

josephwynn-sc commented 5 years ago

Hey, thanks for reporting this! I think WPT already has some mechanism to ignore the status bar, so it should be trivial to implement the same thing for hero times. Failing that, a quick-but-naïve fix would be to ignore the bottom pixels of each frame.

I don't have time to do this at the moment but if somebody wants to have a crack at that, the code is in visualmetrics.py.

github-actions[bot] commented 5 months ago

We're in the process of cleaning up issues on this project in order to ensure we're able to stay on top of high priority bugs and feature requests. As a part of this process, we're automatically closing any issues that have had no activity within the last two years, including this one, since the codebase has changed dramatically in that time. If you feel this is still relevant, please file a new issue using the relevant issue template so we can get it prioritized. Thanks!