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

Wrong committee attendance data for MK Michael Malkieli #132

Closed
shevyk opened this issue Apr 2, 2017 · 6 comments
Closed

Wrong committee attendance data for MK Michael Malkieli #132

shevyk opened this issue Apr 2, 2017 · 6 comments

Comments

@shevyk
Copy link
Contributor

shevyk commented Apr 2, 2017

MK Michael Malkieli does not appear as having attended committee meetings even though his attendance is recorded in the published protocol. See, for example: https://oknesset.org/committee/meeting/13483/

image
image

@alonisser
Copy link
Contributor

@shevyk this is because #133 He can't be in attendance if he is not registered as committee member

@OriHoch
Copy link
Collaborator

OriHoch commented Apr 4, 2017

thanks @alonisser - when I moved the committees scrapers code to knesset-data I don't remember any check of committee memeber.. I'll check again, maybe I missed something

@alonisser
Copy link
Contributor

alonisser commented Apr 4, 2017 via email

@OriHoch
Copy link
Collaborator

OriHoch commented Apr 13, 2017

ok, this is stupid 🤷‍♂️

he had an extra space after his name

"מיכאל מלכיאלי "

  • I edited him on Open Knesset admin - and removed the extra space, so he should appear in future meetings from now on
  • Fixed the scraper that matches attending members to prevent this and similar problems (fixed in v1.8.0)

@alonisser @maryLoi @shevyk

regarding old meetings - I think they will have to be fixed by hand, or in bulk reparse all committee meetings

@OriHoch OriHoch removed their assignment Apr 13, 2017
@shevyk
Copy link
Contributor Author

shevyk commented Apr 23, 2017 via email

@OriHoch
Copy link
Collaborator

OriHoch commented Apr 30, 2017

it's fixed, I rescraped all meetings of the last moshav
also, made sure all MKs don't have spare spaces..

@OriHoch OriHoch closed this as completed Apr 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants