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

InvalidProtoExpectation #39

Closed
davidosomething opened this issue Jan 25, 2017 · 6 comments · Fixed by #103
Closed

InvalidProtoExpectation #39

davidosomething opened this issue Jan 25, 2017 · 6 comments · Fixed by #103
Labels

Comments

@davidosomething
Copy link

davidosomething commented Jan 25, 2017

davidosomething@potatopro:~                                                                                                                                               [js:v6.9.1|py:3.6.0|rb:2.4.0]
 I  % pwmetrics http://elitedaily.com/?omgplzno=11
Launching Chrome
Invalid ProtoExpectation: ignored(2560 2560 undefined) File a bug with this trace!
Invalid ProtoExpectation: ignored(2560 2560 undefined) File a bug with this trace!
Invalid ProtoExpectation: ignored(2560 2560 undefined) File a bug with this trace!
Invalid ProtoExpectation: ignored(2560 2560 undefined) File a bug with this trace!

                       |
First Contentful Paint |                                                               2,349
                       |
First Meaningful Paint |                                                                    2,557
                       |
Perceptual Speed Index |                                                                                                                   4,317
                       |
   Time to Interactive |                                                                                                                                            5,286
                       --------------------------------------------------------------------------------------------------------------------------------------------------------------
                        0                                                              Time (ms) since navigation start                                                               6000

and again 2nd run

pwmetrics "http://elitedaily.com/?omgplzno=11"
Launching Chrome
Invalid ProtoExpectation: ignored(1819 1819 undefined) File a bug with this trace!
Invalid ProtoExpectation: ignored(1819 1819 undefined) File a bug with this trace!
Invalid ProtoExpectation: ignored(1819 1819 undefined) File a bug with this trace!
Invalid ProtoExpectation: ignored(1819 1819 undefined) File a bug with this trace!

                       |
First Contentful Paint |                                                    1,607
                       |
First Meaningful Paint |                                                          1,814
                       |
Perceptual Speed Index |                                                                                                                  3,583
                       |
   Time to Interactive |                                                                                                                                                 4,561
                       --------------------------------------------------------------------------------------------------------------------------------------------------------------
                        0                                                              Time (ms) since navigation start                                                               5000
@denar90
Copy link
Collaborator

denar90 commented Jan 25, 2017

@davidosomething thanks for reporting it. Can you provide us with your operations system name and pwmetrics version. Also do you run it in a terminal or something?

@paulirish
Copy link
Owner

Thanks. It's an upstream bug in Lighthouse I just filed there: GoogleChrome/lighthouse#1534

@davidosomething
Copy link
Author

thanks @paulirish

@pedro93
Copy link
Collaborator

pedro93 commented Mar 8, 2017

@davidosomething Can you check if this bug is still reproducible using the master branch version of pwmetrics?

@davidosomething
Copy link
Author

davidosomething commented Mar 8, 2017

using master, yields
Invalid ProtoExpectation: ignored(4302 4302 undefined) File a bug with this trace!

# history

npm rm -g pwmetrics
hub clone paulirish/pwmetrics
cd pwmetrics
npm install
npm link
rehash
which pwmetrics
pwmetrics http://elitedaily.com/?omgplzno=11
david@elitedavid:~projects/pwmetrics                                                                                                                   [js:v6.9.1|py:3.6.0|rb:2.4.0]
 I  (master)% pwmetrics http://elitedaily.com/?omgplzno=11
Launching Chrome
Invalid ProtoExpectation: ignored(4302 4302 undefined) File a bug with this trace!

                       |
   Time to Interactive |                                                                                                                                       6,642
                       |
Perceptual Speed Index |                                                                                                               5,474
                       |
First Meaningful Paint |                                     1,778
                       |
First Contentful Paint |                                     1,778
                       ---------------------------------------------------------------------------------------------------------------------------------------------
                        0                                                     Time (ms) since navigation start                                                      7000

@pedro93
Copy link
Collaborator

pedro93 commented Mar 15, 2017

As part of the work in progress #71 I've been running pwmetrics with the multiple run options and caught this exception.
This bug leads the program into what I assume is an infinite loop with a deadlock (ctrl+c will not work and interrupt the process), eventually leading to the following output:

screen shot 2017-03-14 at 13 56 54

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants