<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>273363</bug_id>
          
          <creation_ts>2011-05-15 20:09:48 +0000</creation_ts>
          <short_desc>Rating in statistics doesn&apos;t show halves of score</short_desc>
          <delta_ts>2012-07-03 19:03:20 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>amarok</product>
          <component>Collection Browser</component>
          <version>2.5.0</version>
          <rep_platform>Unlisted Binaries</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.6</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="karaluh">karaluh</reporter>
          <assigned_to name="Amarok Bugs">amarok-bugs-null</assigned_to>
          <cc>aakashrajdahal</cc>
    
    <cc>ralf-engels</cc>
          
          <cf_commitlink>http://commits.kde.org/amarok/6014811fda5a1ea71b4576630a7b8ae2e3754e4c</cf_commitlink>
          <cf_versionfixedin>2.6</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1119723</commentid>
    <comment_count>0</comment_count>
    <who name="karaluh">karaluh</who>
    <bug_when>2011-05-15 20:09:48 +0000</bug_when>
    <thetext>Version:           2.4.1
OS:                Linux

2.4.1 regression.

Reproducible: Always

Steps to Reproduce:
Have album with only one track. Give this track 3,5 rating. Edit track information for the whole album. 

Actual Results:  
The album has 3,0 rating.

Expected Results:  
The album has 3,5 rating.

OS: Linux (i686) release 2.6.38-9-generic
Compiler: cc</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1119729</commentid>
    <comment_count>1</comment_count>
    <who name="karaluh">karaluh</who>
    <bug_when>2011-05-15 20:32:37 +0000</bug_when>
    <thetext>Ok, it&apos;s not entirely like this. Rounding algorithm was changed in 2.4.1. For example album with ratings:
3
3,5
4
4
3,5
5
3
4
has now rating 3,5, in all previous versions it had 4, which is mathematically more correct.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1119774</commentid>
    <comment_count>2</comment_count>
    <who name="Myriam Schweingruber">myriam</who>
    <bug_when>2011-05-15 22:52:11 +0000</bug_when>
    <thetext>Not a regression, but a duplicate of an old bug you reported yourself :)

*** This bug has been marked as a duplicate of bug 268246 ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1119858</commentid>
    <comment_count>3</comment_count>
    <who name="karaluh">karaluh</who>
    <bug_when>2011-05-16 08:50:20 +0000</bug_when>
    <thetext>That&apos;s not a duplicate. Both are my reports and both are about different
issues. The 268246 one is about incorrectly saving ratings to files, this one is about incorrectly calculating album rating.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1137397</commentid>
    <comment_count>4</comment_count>
    <who name="Ralf Engels">ralf-engels</who>
    <bug_when>2011-07-01 14:32:18 +0000</bug_when>
    <thetext>For clarification:
With &quot;album rating&quot; you mean the rating that is displayed on the first page of the edit-track-details dialog?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1142934</commentid>
    <comment_count>5</comment_count>
    <who name="Myriam Schweingruber">myriam</who>
    <bug_when>2011-07-18 06:46:22 +0000</bug_when>
    <thetext>Karaluh, could you please answer? Without your feedback there is not much we can do here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1142962</commentid>
    <comment_count>6</comment_count>
    <who name="karaluh">karaluh</who>
    <bug_when>2011-07-18 07:24:08 +0000</bug_when>
    <thetext>Sorry, I forgot about the notification e-mail. Yes, this is the rating indeed. I was refering to it as &quot;album rating&quot; because it is how I use this info.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1142978</commentid>
    <comment_count>7</comment_count>
    <who name="Myriam Schweingruber">myriam</who>
    <bug_when>2011-07-18 08:30:12 +0000</bug_when>
    <thetext>Thank you for the feedback.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1143758</commentid>
    <comment_count>8</comment_count>
    <who name="Ralf Engels">ralf-engels</who>
    <bug_when>2011-07-20 09:11:17 +0000</bug_when>
    <thetext>Seems to work for me.
Can you check it with the latest version and try to give me a more detailed instruction how to reproduce?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1152589</commentid>
    <comment_count>9</comment_count>
    <who name="karaluh">karaluh</who>
    <bug_when>2011-08-11 12:45:28 +0000</bug_when>
    <thetext>Bug still present in 2.4.3. Take 8 tracks, rate 5 of them 3,5 and the rest 3. When you see the track details for all the tracks, the rating is 3, but it should be 3,5 instead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1152619</commentid>
    <comment_count>10</comment_count>
    <who name="Ralf Engels">ralf-engels</who>
    <bug_when>2011-08-11 13:17:01 +0000</bug_when>
    <thetext>Ok.

(5 * 3,5 + 3 * 3) / 8 = 3,3125

That is displayed as 3 stars. So it&apos;s always rounding down.

Is that what you are complaining about?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1154312</commentid>
    <comment_count>11</comment_count>
    <who name="karaluh">karaluh</who>
    <bug_when>2011-08-16 13:29:02 +0000</bug_when>
    <thetext>Yes, that would be the problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1207009</commentid>
    <comment_count>12</comment_count>
    <who name="Aakash">aakashrajdahal</who>
    <bug_when>2011-12-31 03:54:35 +0000</bug_when>
    <thetext>In Amarok 2.5 I am getting the same results as Comment #9. Though not sure whether it is a &quot;bug&quot; or the way it should really be.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1207119</commentid>
    <comment_count>13</comment_count>
    <who name="Myriam Schweingruber">myriam</who>
    <bug_when>2011-12-31 12:50:17 +0000</bug_when>
    <thetext>Setting status to confirmed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1272186</commentid>
    <comment_count>14</comment_count>
    <who name="Matěj Laitl">matej</who>
    <bug_when>2012-07-03 19:03:20 +0000</bug_when>
    <thetext>Git commit 6014811fda5a1ea71b4576630a7b8ae2e3754e4c by Matěj Laitl.
Committed on 03/07/2012 at 18:12.
Pushed by laitl into branch &apos;master&apos;.

TagDialog: correct rounding for rating

I still wonder how observant our users are. :-)
FIXED-IN: 2.6

M  +1    -0    ChangeLog
M  +2    -1    src/dialogs/TagDialog.cpp

http://commits.kde.org/amarok/6014811fda5a1ea71b4576630a7b8ae2e3754e4c</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>