-
Notifications
You must be signed in to change notification settings - Fork 280
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
Added example for Legal Simplifier #141
base: main
Are you sure you want to change the base?
Conversation
@g-linville is going to give this a review |
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.
Thanks for this example! Just a few suggested changes.
I've fixed all the review comments, @cjellick @g-linville |
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.
Found a couple duplicate lines in the script, but other than that LGTM
## Usage | ||
|
||
1. Open your web browser and navigate to `http://127.0.0.1:5000/`. | ||
2. Use the web interface to upload an a legal document in .pdf format. |
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.
2. Use the web interface to upload an a legal document in .pdf format. | |
2. Use the web interface to upload a legal document in .pdf format. |
|
||
Get the part of legal document at index $index. | ||
Do not leave out any important points. Focus on key points, implications, and any notable clauses or provisions. | ||
Do not leave out any important points focusing on key points, implications, and any notable clauses or provisions. |
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.
This line should be removed
Do not leave out any important points. Focus on key points, implications, and any notable clauses or provisions. | ||
Do not leave out any important points focusing on key points, implications, and any notable clauses or provisions. | ||
Give a list of all the terms and explain them in one line before writing the summary in the document. | ||
Give a list of all the terms and explain them in one liner before writing the summary in the document. |
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.
Same here
For each summary write in smaller chunks or add bullet points if required to make it easy to understand. | ||
Use the heading "Summary" only once in the entire document. | ||
Explain terms in simple language and avoid legal jargon unless absolutely necessary. | ||
Explain terms in simple language and avoid legal terminologies until unless absolutely necessary. |
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.
And here
Adding example for Legal Simplifier, an app that simplifies legal documents. For larger documents, this has the ability to slipt them into chunks to fit within the token limit.
This PR also contains the
legal.pdf
file and the generatedsummary.md
file.