-
Notifications
You must be signed in to change notification settings - Fork 1
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
Extend the citation lookup page to use the features of the API #24
Comments
A couple design starting points...
(https://www.courtlistener.com/c/abb-pr/14/)
(https://www.courtlistener.com/opinion/108713/roe-v-wade/authorities/) I think example 1 looks a lot better, and that it can probably be improved further, but these are good places to begin. |
@cmaczo, if you want to spend a little time coming up with a design for this, I think that'd be great. |
Additional observation and questions: If a citation is not found, that needs to be really prominent. How about running it against a PDF or Word doc instead of a blob of pasted text? Maybe that's a premium feature, but people will probably ask for it. In that vein...how crazy would it be to run this on everything in the RECAP Archive that looks like a motion? It would be further interesting to compare those results pre- and post-LLM era. |
Well, I'm not sure we want this to be a big priority that we put a ton of time into, but...
Pretty easy if we've got the rest in place, honestly. We already have infra to do text extraction from documents.
We've already found all the citations across all RECAP documents, but I haven't analyzed the various statuses of those citations. It'd take a few weeks if we wanted to do it. What outcome are you hoping for? |
Working on it! Think I will have some questions for you tomorrow. |
I am wondering if there are hallucinated citations that nobody knows about yet. |
Ah ha. Yeah, that could be quite interesting. if we did it, we'd probably learn which citations we don't have, which would be useful too (even in an ongoing way, that could be useful). I'm not sure we have anybody to do this work though, at least for the moment... |
@mlissner You can find my first draft wireframe for the search and results page here: https://www.figma.com/file/HlrsN7b5wuAQXKMsij1tw5/Free-Law-Project?type=design&node-id=1011%3A7509&mode=design&t=HOFbpGYg91GLfDuT-1 Let's discuss during our meeting tomorrow. I am sure I have misunderstood something. :) |
Nice. A few things:
I think that's it for this round. Want to make a few revisions and I'll take another look? Thank you! |
Ok.
I've added the additional button, but I wonder if people will understand the difference between "Analyze citations" and "Look up citation." I'm not sure I would, just from the button titles.
I don't like them, but they're usually considered best practice where someone could be pasting a huge chunk of text into a text area. For many people, it's hard to delete all that text once it's in there, so must of the UI guides say you should provide clear buttons for them.
Done.
Updated with this new understanding. I think what confused me was this part of your example API output:
Done. |
Looks good to me. I made a few more comments in Figma, but I think it should be a solid place to begin.
There's actually a better way to do this, I think. If there is one citation in the text, we take the user to it. If there's more than one, we provide the page you mocked up. Simple, and that means we only need one button. Perhaps it should just say, "Analyze"? |
It would be weird to be taken to a case if you thought there might be more
than one citation in your text. I’d rather see the result set of one first.
…On Fri, Apr 12, 2024 at 12:42 PM Mike Lissner ***@***.***> wrote:
Looks good to me. I made a few more comments in Figma, but I think it
should be a solid place to begin.
I wonder if people will understand the difference between "Analyze
citations" and "Look up citation." I'm not sure I would, just from the
button titles.
There's actually a better way to do this, I think. If there is one
citation in the text, we take the user to it. If there's more than one, we
provide the page you mocked up. Simple, and that means we only need one
button. Perhaps it should just say, "Analyze"?
—
Reply to this email directly, view it on GitHub
<#24>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAP2Y7CWGKDEEBFQ6OXNNTLY5A2JVAVCNFSM6AAAAABF5OPQMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJSGQZDGMRWGY>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
If we do that, we break compatibility with people that use this URL as an API and who expect a redirect response when their case is found, a 404 when it's not, etc. But aside from that sort-of-requirement, if there's more than one citation in the text, I think we'd find that. When would we miss that and send you to a response when there was more than one citation? |
Our new citation lookup API is really neat. It takes a blob of text, looks up the citations, and tells you what it finds:
Unfortunately, a member of our board, upon seeing it, had a good idea:
So this issue is to jot down some notes about it. Here's how the tool currently looks:
A few changes:
This should now be a
textarea
to encourage people to paste more.At the bottom, it should get a second button that says "Analyze Citations"
The text at the top currently says:
That can change to:
If somebody clicks "Look it Up" we do. Just like now.
If somebody clicks "Analyze Citations" we basically just run it against the API, either internally or via an API call if that seems worth it (I'm doubtful!).
When the response comes back, we show a page that sort of mirrors the current API response. It'll show a list of citations along with:
Here's an API response I put together by hand that kind of shows the types of responses we can expect:
The text was updated successfully, but these errors were encountered: