-
Notifications
You must be signed in to change notification settings - Fork 470
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
NV: update bills scraper to find votes #4763
NV: update bills scraper to find votes #4763
Conversation
@jessemortenson Please review it and let me know if you have issues. |
Hi @braykuka - I ran this and ran into an error about 2 hours into the scrape run:
I should have specified a session, looks like it was going through both the regular and a couple special sessions. |
Also, checked the data from the part of the scrape that I did get, and found a data problem: it looks like some VoteEvents are showing the I'm attaching the JSON output from the scrape, as far as it got. Here are my detailed notes on those examples: BILL: AB215 (session: 82)vote_event_ffc17986-a521-11ee-ad8b-6ff4fa552a1a.json
vote_event_0099bba2-a522-11ee-ad8b-6ff4fa552a1a.json
BILL: AB1 (session: 2023Special34)vote_event_35f19732-a51a-11ee-ad8b-6ff4fa552a1a.jso
vote_event_35f19733-a51a-11ee-ad8b-6ff4fa552a1a.json
Could you work on fixes for that data issue and the error above? |
@jessemortenson I will fix them. |
@jessemortenson |
@jessemortenson I've fixed the issue. Please review it again. Thanks. |
Both of the senate votes. There should be one Vote Event yielded for each vote that occurs on a bill. So for the bill on the URL you cite, there should be 3 Vote Events (two associated with the Senate/upper and one associated with the House/lower). I'll take another look at the code |
I was able to run the full scrape, and the issues remarked upon above are fixed. Thanks! |
Description
I've updated NV bill scraper in order to get votes.