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

Dealing with vote explanations #123

Open
joepio opened this issue Dec 23, 2016 · 2 comments
Open

Dealing with vote explanations #123

joepio opened this issue Dec 23, 2016 · 2 comments
Labels
New property ↩️. Requires discussion

Comments

@joepio
Copy link

joepio commented Dec 23, 2016

In Dutch parliament, both individuals and parties can give an explanation to why they voted in a certain way.

In issue #57, @jpmckinney coined a possible 'note' attribute for votes. I feel like it would be better to have a more descriptive 'VoteExplanation' name. I also think that it should be its own resource, since it can be linked to both votes by individuals and votes by groups (counts).

But perhaps there is a better way of dealing with these explanations. What are your thoughts on this?

@jpmckinney
Copy link
Member

@joepsz I'm not clear on why the explanation needs to be its own object. Can you explain? Vote already allows votes by individuals or groups. If we want to add a note to the Count class, that's easy, too.

@jpmckinney jpmckinney added New property ↩️. Requires discussion labels Dec 24, 2016
@joepio
Copy link
Author

joepio commented Jan 4, 2017

You are right, the property / attribute could be shared between Vote and Counts.

However, I'm a bit confused about using Vote for groups (specifically political parties). It creates problems when doing calculations on votes, since the vote of a party has a certain weight. Because of this, we use a Count for each political party (and each option, if its members voted differently). And if we do that, we still need a VoteExplanation (or Note) property on Counts as well as Votes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New property ↩️. Requires discussion
Projects
None yet
Development

No branches or pull requests

2 participants