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

Cookbook#08 #80

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

Conversation

duper203
Copy link
Contributor

@duper203 duper203 commented Oct 8, 2024

approval typically needed for code added to solve context length error

pro model의 context length 한계로 retrieval 이후의 임의적인 코드 수정이 있습니다.
context length 에러로 이것저것 시도(chunking without using db, summarization etc.) 해 본 결과, chain.invoke 답변이 제일 좋게 나온것으로 수정
+
마지막에 더 precise한 답변을 위한 다른 방법 추가

2) DocParse
3) solar pro
4) change of code in the "querying"
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

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.

1 participant