Closed
Bug 830902
Opened 12 years ago
Closed 10 years ago
Adjust Popularity measure
Categories
(Marketplace Graveyard :: Statistics, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
2015-02-03
People
(Reporter: clouserw, Assigned: robhudson)
References
Details
(Whiteboard: [waiting on monolith][feature][better-search])
Due to differences in hosted vs packaged (hosted doesn't re-ping the market) we can't base our stats on active daily users. We should base the popularity of apps solely on average number of installs over the last 30 days.
Reporter | ||
Updated•12 years ago
|
Target Milestone: 2013-01-24 → 2013-01-31
Reporter | ||
Comment 1•12 years ago
|
||
Let's do this with monolith
Component: General → Statistics
Whiteboard: [waiting on monolith]
Target Milestone: 2013-01-31 → ---
Assignee | ||
Comment 2•11 years ago
|
||
Popularity uses the `Installed` table which only includes logged-in user installs. We should include anonymous installs when calculating popularity as well.
This would likely need a new table since we want popularity per region and don't want to have to pull this from monolith whenever we want it. (And to avoid coupling ES indexing to require Monolith)
Assignee | ||
Comment 3•11 years ago
|
||
Note: We recently updated the code to store `weekly_downloads` and `total_downloads` which is populated via Monolith. Currently popularity uses `Installed` which is total installs -- albeit not completely reliable and only counts logged-in user installs.
We could pretty simply switch to using `total_downloads` as the popularity metric.
Reporter | ||
Comment 4•11 years ago
|
||
let's do it
Comment 5•11 years ago
|
||
Should we do total downloads over last X days (like 90?) so that newer, more popular things have a chance to make the popular list?
Updated•11 years ago
|
Whiteboard: [waiting on monolith] → [waiting on monolith][feature]
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Comment 6•10 years ago
|
||
I filed bug 1116564 to fix or remove regional popularity. If we moved to calculating popularity as "installs in the last 90 days" we could also calculate that per region and make regional popularity actually mean something.
Updated•10 years ago
|
Whiteboard: [waiting on monolith][feature] → [waiting on monolith][feature][better-search]
Assignee | ||
Comment 7•10 years ago
|
||
https://github.com/mozilla/zamboni/commit/7388ab8
To verify:
* Ensure that a re-index has occurred to pull in the new popularity score (ask robhudson on IRC)
* View apps sorted by popularity.
* Ensure that the top apps look reasonable based on their 90 day install counts in statistics.
Assignee: nobody → robhudson.mozbugs
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2015-02-03
Comment 8•10 years ago
|
||
Verified as fixed in FF38(Win7)
I have checked statistics for several apps in the Popular list and they look good. The apps with most install counts are displayed first.
Closing bug.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•