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

Added example for Legal Simplifier #141

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

techmaharaj
Copy link
Contributor

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 generated summary.md file.

@cjellick
Copy link
Contributor

@g-linville is going to give this a review

Copy link
Member

@g-linville g-linville left a 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.

examples/legalsimplifier/README.md Outdated Show resolved Hide resolved
examples/legalsimplifier/README.md Outdated Show resolved Hide resolved
examples/legalsimplifier/legalsimplifier.gpt Show resolved Hide resolved
examples/legalsimplifier/legalsimplifier.gpt Show resolved Hide resolved
examples/legalsimplifier/legalsimplifier.gpt Show resolved Hide resolved
examples/legalsimplifier/legalsimplifier.gpt Outdated Show resolved Hide resolved
examples/legalsimplifier/legalsimplifier.gpt Show resolved Hide resolved
examples/legalsimplifier/summary.md Outdated Show resolved Hide resolved
@techmaharaj
Copy link
Contributor Author

I've fixed all the review comments, @cjellick @g-linville

Copy link
Member

@g-linville g-linville left a 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.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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.
Copy link
Member

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.
Copy link
Member

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.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And here

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

Successfully merging this pull request may close these issues.

3 participants