Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Participant estimation #745

Open
ossie opened this issue Sep 23, 2020 · 1 comment
Open

Participant estimation #745

ossie opened this issue Sep 23, 2020 · 1 comment
Assignees

Comments

@ossie
Copy link

ossie commented Sep 23, 2020

Current behaviour
A second participant number estimation by the same user is counted as a new estimation and calculated into the average.

Expected behaviour
The second estimation should be considered as a correction of the first one.
e.g.: I estimated 50 and after counting the participants in a video I entered 62: Now the number, based on my estimation only, is 56.

@maltehuebner
Copy link
Contributor

Thank you for addressing this issue.

I got many questions regarding participant estimation after the kidical mass last weekend. I guess there is a lot of work waiting for me as the estimation system is one of the oldest parts of the critical mass application.

At the moment, the system grabs all estimations assigned to a ride and calculates the average. This isn’t even kind of transparent, as it leads to rough estimates.

https://github.com/criticalmass-one/criticalmass-in/blob/master/src/Criticalmass/Statistic/RideEstimateCalculator/RideEstimateCalculator.php

I would prefer to add a list to the ride overview page where all estimates are presented in a table and the community can pick the most accurate. Additionally, it should be possible to add a coordinate to the estimate to see where the bikes were counted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants