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

Implement query to get block theme adoption based on new HTTP Archive custom metric #53

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

felixarntz
Copy link
Collaborator

@felixarntz felixarntz commented May 8, 2023

This query has the same purpose as the one implemented in #32, however it uses the HTTP Archive custom metric added in HTTPArchive/custom-metrics#62, which has been available in HTTP Archive datasets starting with March 2023.

This custom metric makes the query (which was extremely slow before as it had to parse entire response bodies) notably faster (in a comparison 3:33 minutes vs 35:49 minutes).

Query results

Row client total_wp_sites with_block_theme pct_total
1 desktop 4519655 31102 0.0068814986984626035
2 mobile 5829541 43403 0.00744535461711308

Based on April 2023 dataset

Comparison with old query

For reference, to assess the new numbers and performance of the query, the query from #32 was also run again for the April 2023 dataset. Here are the results:

Row client with_block_theme total_wp_sites pct_total wp_gte_59  
1 desktop 12669 4506581 0.002811222077224397 3810151
2 mobile 19992 5825503 0.0034318066611587015

This shows slightly lower percentages, likely because the query's approach of parsing the HTML from the response bodies is more error-prone.

Copy link
Collaborator

@mukeshpanchal27 mukeshpanchal27 left a comment

Choose a reason for hiding this comment

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

Thanks @felixarntz, PR look good to me. I have updated table structure for old data.

Note: For old query the mobile data for wp_gte_59 is missing.

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

Successfully merging this pull request may close these issues.

2 participants