-
Notifications
You must be signed in to change notification settings - Fork 499
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
Determine how to associate Congress number with terms #18
Comments
I just realized that I was conflating the yaml representation with the XML representation, especially in my comment in #15. Clearly, it is not a big deal to just have a list of Congresses associated with a term. (Potentially allowing representatives' terms to use a single string/int instead of a list.) |
We'd probably want to have it be the same data type (string or array) for both reps and sens, to keep parsing saner, but yeah, an array would handle the ambiguity. |
Oh, right, because Resident Commissioners are listed under "rep". So list it is! |
Well, actually, I think we should change the title for Resident Commissioners to "Com", and Delegates to "Del". (These are the actual titles they use.) We added a "chamber" field at my suggestion so that titles could diversify more easily. For example, it's possible that someday the Senate might get delegates - a Senate committee came within one vote of passing a bill that would have given PR a non-voting Senate delegate: I think Josh objected earlier because it would require some GovTrack work, so we might want to put Com/Del titles in the little bucket of things to wait a few weeks on. But either way, a list of one congress for House members isn't the end of the world, it's worth it for consistency. |
Let's punt the question of chamber vs title to another issue. For that, someone's going to have to do some research for the historical data. But I'm ok with whatever you guys decide for adding congresses to each role. |
Good point on the historical data. What is this issue trying to solve, btw? When would someone need to On Tue, Jan 8, 2013 at 7:50 PM, Joshua Tauberer notifications@github.comwrote:
Developer | sunlightfoundation.com |
I've spun off the issue on chamber/types into #26 (though I suppose it could be debated as whether a third issue is needed to discuss titles). As for a Congress: It would be a way to short-circuit the lookup of a Congress from the dates of a term. I'm not sure there's an immediate usecase right now, but it's an important bit of information to have, especially as we get further and further away from the date-Congress mappings we're familiar with. (Like, if I said the person served in 1873, could you tell me what Congress that was without having to do a bunch of math?) |
Oh, one usecase might be to answer this question: What Congresses did this person serve in? Then all you'd have to do is look at the congress field in all their terms. |
When updating an existing db from the YAML, matching terms to existing records is not really well defined without a primary key or some other constraint. Congress numbers would help that. (This was my major issue last week.) |
Well I'm certainly not opposed to it, if it's easy enough to keep up to On Wed, Jan 9, 2013 at 12:35 PM, Joshua Tauberer
Developer | sunlightfoundation.com |
This seems to have stalled - let's re-open it when we have a plan of action. |
This should probably be reopened, since @dwillis is working on it. |
Nah, then we'd have two issues open for it. :) I can't handle all these On Thu, Mar 7, 2013 at 12:55 PM, Gordon P. Hemsley <notifications@github.com
Developer | sunlightfoundation.com |
What's the second one? #41 is for how someone got into their position, which is a completely separate issue. |
Derek's doing both at the same time, I think, so he can close it out all at On Thu, Mar 7, 2013 at 1:40 PM, Gordon P. Hemsley
Developer | sunlightfoundation.com |
There are actually two separate issues here which may or may not be solved with a single solution:
I mentioned a few possible solutions in #15, some of which were controversial.
The text was updated successfully, but these errors were encountered: