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.

Sign up for GitHub

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

Jump to bottom

Fix: Add a text-splitter, fix the graph's tool router #39

Closed
wants to merge 101 commits into from
Closed

Fix: Add a text-splitter, fix the graph's tool router #39

wants to merge 101 commits into from

Conversation

palaniappan-r
Copy link
Collaborator

@palaniappan-r palaniappan-r commented Jul 15, 2024 •
edited
Loading

  • Fixed the agent function in retriever_graph.py
  • Added a text-splitter post the document chunking

palaniappan-r and others added 30 commits July 1, 2024 23:37
@palaniappan-r
feat: rewrite md loading
73c5a08 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: stop returing retrievers
82609f7 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: add multi retriever chain
637d0c4 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: refactor markdown loading and processing
7a289df 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
style: ruff format edits
cceaec4 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
reafctor: update fn names
c5b6e86 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: change fn name
4e71a62 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: set up api structure
ef9f739 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: add api
86bce02 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: add chain router
f70b30a 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: update uvicorn to use the api
0c17152 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: remove unused imports, returns
3d967e3 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: add module to run as a main method
fb1b220 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
reafactor: change default chunk size
68e1998 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: Add hybrid, similarity, and multi retriever chains
c87d8b5 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: rename file
3f753c8 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: remove server.py
85def3e 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
chore: add dev requirements
b96728b 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
chore: remove dev deps
a30a10a 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: add list chains endpoint
446f46c 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: removed unused file
80246d1 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: change api path in Dockerfile
2fcb94f 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: rename requirements
92572aa 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
chore: add env var for use_cuda
c35ef8c 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
style: rename LangchainDocument to Document
f029ba9 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: store prompt templates in a seperate file
d7cc4ec 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
style: rename LangchainDocument to Document
b55e0bd 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: load prompt templates from a sep folder
0804701 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
style: rename requirements
e666d88 
Signed-off-by: Palaniappan R <[email protected]>
@error9098x
Added Dropdown for Multiple Endpoints
738a996 
- Added support for multiple endpoints via dropdown to choose the preferred architecture.

- Added Response time in sec ( text color indication: below 5 sec green, below 10 sec orange, above 10 sec red )

Signed-off-by: ProCodec <[email protected]>
@palaniappan-r
feat: get embeddings, reranker and use_cuda from env vars
1221a42 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: update correct types, remove main entry point
56f7ac3 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: print an error message when the source url is absent
09dfd38 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feart: add a check for env vars
87ba4c9 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: add use_cuda var
313c402 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
chore: change default use_cuda value
5256489 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: use a single fstring
e7c8616 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: parse use_cuda env var properly
4bfe11d 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: add an init function
6002ad7 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
style: ruff formatting fixes
57cf5b5 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: remove unused imports
60b4036 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: remove main entry
05eb137 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: throw error if src is null
d7dae1e 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: throw error is query is empty
d41b4fb 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
Merge branch 'master' into agent-graph
9760687 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: correct typo in requirements.txt
5f82149 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
chore: update requirements
c4f80c5 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: mypy typing checks
999e757 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: use the built in list instead of List
b5d6335 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
refactor: Update RetrieverAgent and HybridRetrieverChain signatures
ac3f462 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: mypy type check fixes
b1f8e7b 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
style: switch to single quotes
ac88411 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
fix: return empty tools list only when the tool_calls param is null
faa4372 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
feat: add RecursiveCharacterTextSplitter
232c9a7 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r
chore: Update .env.example with Google Application Credentials path
f89c373 
Signed-off-by: Palaniappan R <[email protected]>
@palaniappan-r palaniappan-r changed the title Fix: Add a text-splitter, fix the graph's tool node Fix: Add a text-splitter, fix the graph's tool router Jul 15, 2024
@luarss
Copy link
Collaborator

luarss commented Jul 15, 2024

Hi @palaniappan-r can you recreate a new PR from a new branch? This PR still uses agent-graph

@luarss luarss closed this Jul 15, 2024
@luarss luarss deleted the agent-graph branch August 8, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luarss luarss Awaiting requested review from luarss

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@palaniappan-r @luarss @error9098x