Ratings and Reviews server

Review statistics are language/distroseries dependent

Bug #686553 reported by Michael Nelson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ratings and Reviews server
Fix Released
Medium
Danny Tamez
Ratings and Reviews server 11.01

Bug Description

From a conversation last week (with Danny and Anthony) I'd understood that we wanted to return the rating statistics (average rating and number of ratings) in a language-agnostic, and distroseries agnostic way.

Currently the statistics returned by ReviewsStatsHandler.read() are aggregates for the specified language and distroseries.

We chatted about storing these aggregates instead of calculating them, with a cached property on SoftwareItem which would update itself whenever a new review was added (ie. it would re-calculate the count and average based on the DB, rather than incrementing which would not be safe).

Unanswered:
 * Do we also want to store these per distroseries?
 * Comment 2 implies that we *do* want to also store ratings per software-item-release (something that we're not yet doing), so that we can use the two in a weighted calculation.

See original description
Revision history for this message
Michael Nelson (michael.nelson) wrote :

Actually, if we do this, we should change the URL also.

Danny Tamez (zematynnad)
Changed in rnr-server:
assignee: nobody → Danny Tamez (zematynnad)
status: Confirmed → In Progress
Revision history for this message
Matthew Paul Thomas (mpt) wrote :

It's fairly likely that people will rate software differently in different languages, particularly if the software is poorly localized.

It's possible that people will rate the same version of the same software differently on different OS versions, e.g. if the software relies on a library that's crashier in a newer OS version.

Initially, though, I think we will have so few reviews that we should divide them the minimum necessary for fairness. So the median rating should be based on what version of the item you're using, not on what language you're using or what OS version you're using.

Michael Nelson (michael.nelson)
description: updated
Danny Tamez (zematynnad)
Changed in rnr-server:
status: In Progress → Fix Committed
Anthony Lenton (elachuni)
Changed in rnr-server:
milestone: none → 11.01
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.