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

Video bytes and requests not displayed for a single selected month #619

Open
screenspan opened this issue Jul 7, 2022 · 3 comments
Open
Labels

Comments

@screenspan
Copy link

screenspan commented Jul 7, 2022

TESTED ON:
Google Chrome 103.0.5060.114 on Mac OS 11.6.7

STEPS TO REPRODUCE ERROR:

  1. Go to the Page Weight report for a single month (e.g. https://httparchive.org/reports/page-weight?start=2022_01_01&view=list#bytesVideo)

CURRENT RESULT:

  • Median video bytes transferred for a single month is displayed as "0".

image

- The median video bytes transferred for **multiple months** is non-zero (e.g. 3490.6 KB median desktop and 3366.4 KB median mobile from January 2022 - June 2022; https://httparchive.org/reports/page-weight?start=2022_01_01&end=latest&view=list#bytesVideo).

image

EXPECTED RESULT:

@screenspan
Copy link
Author

Please let me know if this is really a bug or just a misinterpretation of the data on my part!

@rviscomi rviscomi transferred this issue from HTTPArchive/almanac.httparchive.org Jul 7, 2022
@rviscomi
Copy link
Member

rviscomi commented Jul 7, 2022

Hi @screenspan thanks for filing this issue.

This looks like an inconsistency in the way the bytesVideo timeseries and histograms are queried. In the timeseries query, it only looks at pages having bytesVideo > 0. The histogram query should similarly filter out the 90%+ pages with 0 bytes of video.

Same issue with reqVideo.

@rviscomi rviscomi added the bug label Jul 7, 2022
@max-ostapenko
Copy link
Contributor

Many other histogram charts (e.g. reqCSS or reqJS) have a bucket that shows 0 bytes / requests.
It just happens that for video resources (and bytesOther) it represents 90%+.
And due to this median video size is actually 0KB.
Is it the best insight to offer - maybe not.

The filter in the timeseries leads to permanently higher medians - in my opinion it should be removed to represent complete dataset aligned with histograms.
Only for other and video we can keep it ON, but leave a note in the description.

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

No branches or pull requests

3 participants