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

Issue with the logic of "Sponsors of Related Measures" #326

Closed
ayeshamk opened this issue Apr 23, 2021 · 6 comments
Closed

Issue with the logic of "Sponsors of Related Measures" #326

ayeshamk opened this issue Apr 23, 2021 · 6 comments
Assignees

Comments

@ayeshamk
Copy link
Collaborator

"Sponsors of Related Measures" should have more records than the "Committee & Co-sponsors" Table. The top-right table is expected to show a subset of the lower table list. When the users click on "View Full List", they should see the full list.

However, I noticed for many bills, "Sponsors of Related Measures" has fewer records than "Committee & Co-sponsors".

Attached one example below:

image

When we click on "View Full List" we see a smaller list

image

@aih
Copy link
Collaborator

aih commented Apr 23, 2021

The bottom table should be a super-set of the top table, but they're calculated differently. Per discussion with Daniel (see #319) the bottom table will have not only the sponsors for the current bill (which the top table is drawn from), but also cosponsors of 'identical' bills from this and previous congresses.

@aih
Copy link
Collaborator

aih commented May 21, 2021

This issue no longer applies to the current Cosponsors table logic.

@aih aih closed this as completed May 21, 2021
@aih aih reopened this May 21, 2021
@aih
Copy link
Collaborator

aih commented May 21, 2021

This appears to still be an issue in some bills; here Ed Case (who is a Congressman in the 117th Congress) is not shown.
image
image

@aih
Copy link
Collaborator

aih commented May 21, 2021

Collin C. Peterson is missing here because he is no longer a Member:
image

image

@aih
Copy link
Collaborator

aih commented May 21, 2021

Fix in 6079d3a to add lone sponsors to the Bill-Cosponsor Join table

@aih aih closed this as completed May 21, 2021
@aih
Copy link
Collaborator

aih commented May 21, 2021

Fixed
image

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

2 participants