-
-
Notifications
You must be signed in to change notification settings - Fork 110
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
fix(uscfc): implement new site #1224
base: main
Are you sure you want to change the base?
Conversation
Solves #1221 - Delete `uscfc_vaccine_u` and `uscfc_u` and their example files. Published and unpublished opinions are in the same page for `uscfc`. On `uscfc_vaccine`, we only find Unpublished opinions
I think you need to parse out the vaccine stuff separately from the rest of the federal claims opinions. This case for example is a published decision. |
# claims. This will help disambiguation, in case docket | ||
# number collide | ||
if self.is_vaccine and not docket.lower().endswith("v"): | ||
docket += "V" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think docket numbers should resember if possible the full docket number in the opinion
No. 22-0075V
No. 22-0667V
in this case adding leading zeroes
self.cases.append( | ||
{ | ||
"url": opinion["link"], | ||
"summary": opinion["text"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont think the summary text you identify fits the criteria of what I would classify as a summary (at least for the vaccine court).
summaries: "PUBLIC DECISION (Originally filed: 11/03/2023) regarding [32] DECISION Stipulation/Proffer ( Signed by Chief Special Master Brian H. Corcoran. )(mpj) Service on parties made."
I think its closer to a disposition, but not really even that it's more like a document description. which we dont collect in case law.
if self.is_vaccine and not docket.lower().endswith("v"): | ||
docket += "V" | ||
|
||
judge = judges_mapper.get(opinion["judge"], "") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should parse out the full name instead of using the judge field in the json
4c12aef
to
9080977
Compare
- parse full judge names - implement extract_from_text for uscfc_vaccine - improve docket number for uscfc_vaccine
@flooie I addressed your comments, please check again To get the proper status for vaccine opinions I had to implement About the summary, it has varying amount of information. Once the judges are removed, it reads better. I also picked up the (originally filed... ) and put it into About it holding info for
|
9080977
to
b526cb9
Compare
Solves #1221
uscfc_vaccine_u
anduscfc_u
and their example files. Published and unpublished opinions are in the same page foruscfc
. Onuscfc_vaccine
, we only find Unpublished opinions