Why is it that one of my values in the dictionary is not being divided the same way and how would I fix it?

Asked Jul 05 '16 at 15:22

Active Jul 05 '16 at 15:26

Viewed 35 times

I have a part of my code where the first function adds all the values in a dictionary and another function that divides each value from the overall value which means that the summary should all equal to 100 however I have this situation where the last value does not divide properly:

 dict = {'Cancelled': 31, 'Accepted': 145, 'Expired': 128, 'Rejected': 779}
 overall = 1083.0
 general_status_summary = {'Cancelled': 2.8624192059095108, 'Accepted': 13.745868405204407, 'Expired': 13.858688982866132, 'Rejected': 96.236168061830412}

As you can see, the rejected key is not correct but the other three are right. I am confuse as to why this is. Here is my code below:

 status_summary = statusSummary(2014, 04, 1, 2015, 03, 31)

    def overall(dict):
        overall_number = sum(dict.values())
        return float(overall_number)    

    overallstatus = overall(status_summary)

    def summaryPercentage(dict):
        for key, value in dict.items():
            dict[key] = (value/(overall(dict))) * 100
        return dict

    general_status_summary = summaryPercentage(status_summary)

edited Jul 05 '16 at 15:26

asked Jul 05 '16 at 15:22

peaceandiago

1

This is not a duplicate – Adam Sosnowski Jul 05 '16 at 15:25
Don't recompute total while you modify the dict. – Adam Sosnowski Jul 05 '16 at 15:27
1

This isn't a duplicate. Unmark it please – Brian Jul 05 '16 at 15:27
i don't think it i is a duplicate either, it has a sort of tangent but it doesn't address a solution – peaceandiago Jul 05 '16 at 15:27
This is not caused by floating point weirdness. Not a duplicate. – Jul 05 '16 at 15:28
The problem is not arithmetic, the problem is that each element is divided by different total. – Adam Sosnowski Jul 05 '16 at 15:28
2

@peaceandiago, you can change your summaryPercentage function like so: `def summaryPercentage(d): newDict = dict() for key, value in d.items(): newDict[key] = (value/(overall(d))) * 100 return newDict` – Brian Jul 05 '16 at 15:28
That was formatted terribly.... – Brian Jul 05 '16 at 15:29
@Brian this is the right answer – peaceandiago Jul 05 '16 at 15:32

Why is it that one of my values in the dictionary is not being divided the same way and how would I fix it?

0 Answers0