onaio / tally-ho

Election results data entry and verification software.
http://tally-ho.readthedocs.org/en/latest/
Other
19 stars 21 forks source link

Election Statistics Report #416

Closed JohnMwashuma closed 1 year ago

JohnMwashuma commented 1 year ago

Overview


Election statistics report per election and per ballot.

Context


Implements the report in slide one of this google sheet file https://docs.google.com/spreadsheets/d/1YnykTc6-MYjokM15VRmZcSWnBS-JESA1eTEhDEivVu8/edit#gid=0

Task check list


codecov-commenter commented 1 year ago

Codecov Report

Merging #416 (2f60ec2) into master (d32461c) will decrease coverage by 0.54%. The diff coverage is 42.10%.

@@            Coverage Diff             @@
##           master     #416      +/-   ##
==========================================
- Coverage   70.68%   70.15%   -0.54%     
==========================================
  Files         138      139       +1     
  Lines        9051     9220     +169     
==========================================
+ Hits         6398     6468      +70     
- Misses       2653     2752      +99     
Files Changed Coverage Δ
tally_ho/libs/models/enums/gender.py 100.00% <ø> (ø)
.../tally/views/reports/election_statistics_report.py 41.07% <41.07%> (ø)
tally_ho/apps/tally/views/super_admin.py 68.54% <100.00%> (+0.06%) :arrow_up:
tally_ho/urls.py 96.66% <100.00%> (+0.11%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

peterMuriuki commented 1 year ago

Looks good, Just a few issues.

The %turnout seems to be off, it seems it calculates it as registrants/voters while I think it should be voters/registrants.: Screenshot from 2023-08-16 12-49-11h

Screenshot from 2023-08-16 12-48-46

======

Also, in the below screenshot: image

The total voters are >0, but male voters + female voters = 0, this is objectively wierd, I understand there is the Unisex votes, which if included would make the displayed data be more complete, I however think if I remember correctly the consesus was to split the Unisex between male and female!!!!,