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

core: replace EIL with MP-FID #8729

Merged
merged 2 commits into from
May 1, 2019
Merged

core: replace EIL with MP-FID #8729

merged 2 commits into from
May 1, 2019

Conversation

patrickhulce
Copy link
Collaborator

Summary
Promotes Max Potential FID to the spot of EIL, moves EIL to JSON only.

@patrickhulce: oh we're def making that happen for v5? 😀
@paulirish: ya

It's difficult to analyze our thresholds because the HTTPArchive data we have on max potential FID will all by impacted by the missing microtask data bug (see #8379), but MPFID differentiates *MUCH better than than EIL, they feel about right, and its still not a scored metric anyway...

Ventiles

SELECT
  APPROX_QUANTILES(CAST(JSON_EXTRACT_SCALAR(report, '$.audits.max-potential-fid.rawValue') as FLOAT64),
    20) as mpfid,
  APPROX_QUANTILES(CAST(JSON_EXTRACT_SCALAR(report, '$.audits.estimated-input-latency.rawValue') as FLOAT64),
    20) as eil
FROM
  httparchive.lighthouse.2019_03_01_mobile

image

Related Issues/PRs
picks up the mantle of #5842 and raises it to glory.

@brendankenny
Copy link
Member

but MPFID differentiates *MUCH better than than EIL, they feel about right

that's because one tries to be robust to outliers and the other is "here's your outlier", but w/e :P

Copy link
Member

@brendankenny brendankenny left a comment

Choose a reason for hiding this comment

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

LGTM!

description: 'The potential First Input Delay that your users could experience is the ' +
'duration, in milliseconds, of the longest task.',
description: 'The maximum potential First Input Delay that your users could experience is the ' +
'duration, in milliseconds, of the longest task. [Learn more](https://developers.google.com/web/updates/2018/05/first-input-delay).',
Copy link
Member

@brendankenny brendankenny May 1, 2019

Choose a reason for hiding this comment

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

Not having the "maximum" part of the string in all the translations (until the next round comes out) isn't a huge deal, but how much do we care about getting this link out to everyone? We could manually edit the other languages to add the Learn More link.

Copy link
Member

Choose a reason for hiding this comment

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

We could manually edit the other languages to add the Learn More link.

note: this could also be a really really bad idea :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ha, Given that these are hidden by default, IMO it's probably not the end of the world to update these ASAP, but others could feel differently.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, SG

@patrickhulce
Copy link
Collaborator Author

that's because one tries to be robust to outliers and the other is "here's your outlier", but w/e :P

apologies, didn't intend to be hating on EIL's usefulness as a whole. I know it's by design but it's definitively not as suitable for differentiating perf scores on top half of the web that's all

@paulirish paulirish mentioned this pull request May 1, 2019
67 tasks
@brendankenny brendankenny merged commit ed71c9c into master May 1, 2019
@brendankenny brendankenny deleted the mpfid branch May 1, 2019 19:12
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.

2 participants