Page MenuHomePhabricator
Create Task
Maniphest T219650

Inconsistent Avg. Daily Pageview results between Pages Improved and Event Summary
Closed, ResolvedPublicBUG REPORT
Actions

Description

I found an inconsistency in the Avg. Daily Pageview numbers in the Pages Improved report vs. Event Summary reports. To reproduce:

  • Download the Pages Improved csv for an event. I the testing event "At Least One of All Contribution Types" (In the Program "joe's testing events").
  • Add up the Avg. Daily Pageviews numbers for en.wiki pages only.
  • Compare to the Event Summary onscreen report, which gives a per-wiki breakdown and also shows Avg. Views for en.wiki.
  • Expected results: the numbers should be the same.
  • Actual results: As of 2019-03-28 18:13, Pages Improved shows 13,515 but the Summary number is 12,937—a pretty big difference.

@dom_walden confirms he saw this discrepancy elsewhere as well. It looks like a math problem, maybe?

Related Objects
Search...

Event Timeline

• jmatazzoni created this task.Mar 29 2019, 5:13 PM

I put this directly into the Ready column. It would be nice if we could clear it up today, since we are clearly giving wrong numbers.

MusikAnimal claimed this task.Mar 29 2019, 6:19 PM
MusikAnimal moved this task from Ready to In Development on the Community-Tech-Sprint board.

This one's a doozy!

• aezell added a comment.Mar 29 2019, 6:32 PM

At first glance, it appears to largely use the same functions which is interesting. I'm taking a closer look.

• aezell added a comment.Mar 29 2019, 6:59 PM

As best I can tell, both of those reports end up resolving down to PageviewsRepository::getPageviewsPerArticle() for the source of the data.

I'll ask a naive question: The Pages Improved number is the average pageviews for improved pages only. The Event Summary number is the average pageviews for all pages in the Event. That might include pages that are created but not improved. Isn't it logical that these numbers could be different?

MusikAnimal moved this task from In Development to Needs Review/Feedback on the Community-Tech-Sprint board.Mar 29 2019, 9:24 PM

Finally figured it out, it was getting pageviews since the start of the event and not a 31 day window. PR: https://github.com/wikimedia/eventmetrics/pull/261

MusikAnimal mentioned this in rGEVM6c978584f95f: Fix fetching of average pageviews for Pages Improved report.Mar 29 2019, 9:26 PM
MaxSem moved this task from Needs Review/Feedback to QA on the Community-Tech-Sprint board.Mar 29 2019, 9:56 PM
• jmatazzoni closed this task as Resolved.Mar 29 2019, 10:07 PM
• jmatazzoni moved this task from QA to Q3 2018-19 on the Community-Tech-Sprint board.

The numbers match! Resolving this.

• jmatazzoni moved this task from New & TBD Tickets to In Sprint 🏃‍♀️🏃‍♂️ on the Community-Tech board.Apr 16 2019, 11:07 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits