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

Reports from documents! #429

Closed
wants to merge 8 commits into from

Conversation

proy9714
Copy link
Contributor

@proy9714 proy9714 commented Apr 4, 2024

A highly requested feature!!! 🔥 🚀 💯

Now generate reports from your own documents!!

Just add the env variable DOC_PATH pointing to the folder where your document are located and choose MY DOCUMENTS from the source.

image

Support for a few basic document types, the community integrate many more from the host of document loaders provided by the langchain ecosystem.

There you go @assafelovic !

@assafelovic
Copy link
Owner

assafelovic commented Apr 4, 2024

This is huge @proy9714 I will review it in a few days, feel free to continue refining in the meanwhile 👏🤩

@ElishaKay
Copy link
Contributor

ElishaKay commented Apr 25, 2024

Looks good guys!
I noticed that there is an error running on mac:

docker-compose up --build

The fix is here.

@ElishaKay
Copy link
Contributor

ElishaKay commented Apr 26, 2024

Within the Readme, you'll want to be a little more specific on the limits of the DOC_PATH variable - ideally it should be somewhere within the gpt-researcher project folder.

For example, we should recommend something like:

DOC_PATH=./docs/my-docs

Reason being, when you're running the app via Docker, when you try to set the DOC_PATH variable to your Downloads folder:

  • you'll get a "Permission Denied" error
  • plus some other errors and complexity which arise, like needing to load volumes into Docker, which we should avoid.

@proy9714
Copy link
Contributor Author

Hey @assafelovic I was on vacation and back now. Is there any problems with merging this? Can you quickly get me up to speed?!

@assafelovic
Copy link
Owner

@proy9714 welcome back from vacation :)

@ElishaKay took the liberty of fixing some issues with docker and created a new PR for this. Ping me on Discord and I'll be happy to jump you in!

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.

None yet

3 participants