-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Research with local documents #452
Conversation
Thanks @ElishaKay ! Did you see that there are many conflicts? |
108cdc2
to
7d67727
Compare
Don't we also need to modify the docker-compose to provide the mount path? |
@arsaboo better to include the documents within the repository itself. |
@ElishaKay @arsaboo I think the mounting a docker volume is a good idea. This is being used by many dockerized applications. Maybe it could be optional to keep the default simple. I guess very often production deployments wouldn't like to have their docs in git. Thanks for this work. Can't wait that it's merged. |
…'resource_report' since resource_reports aren't yet supported for Documents source
4d1bd4c
to
8ea6e94
Compare
2-minute internet and documents reports work after merging latest master. |
…th internet & document options :)
@assafelovic,morning sir.
And best of all.... zero conflicts with master 🎉 @proy9714 feel free to leave feedback here as well - solid stuff! |
Thank you @proy9714 and @ElishaKay for this super super demanded feature! I'll stop all PRs until we get this through (except for hot fixes)! I will dive into it on Wed and we should release it next Monday |
@ElishaKay can you update the documentation here: https://github.com/assafelovic/gpt-researcher/blob/master/docs/docs/gpt-researcher/tailored-research.md |
…l pip package is updated with support for verbose keyword
docker-compose.yml
Outdated
build: ./ | ||
environment: | ||
OPENAI_API_KEY: ${OPENAI_API_KEY} | ||
TAVILY_API_KEY: ${TAVILY_API_KEY} | ||
DOC_PATH: "=./docs/my-docs" |
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.
@ElishaKay are you sure we should keep this static here? Probably keep it null as default?
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.
remove this directory
.gitignore
Outdated
@@ -16,6 +16,9 @@ venv.bak/ | |||
#Ignore generated outputs | |||
outputs/ | |||
|
|||
#Ignore my local docs | |||
docs/my-docs/ |
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 shouldn't be in docs. Docs is intended for documentation files
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 is HUGE!
Fixes an error when running on mac:
docker-compose up --build
The error (both on this branch & master branch) is:
Failed to fetch http://deb.debian.org/debian-security/pool/updates/main/f/firefox-esr/firefox-esr_115.3.1esr-1%7edeb11u1_arm64.deb 404 Not Found [IP: 199.232.82.132 80]
#0 2.622 E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?