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

Gestion fine des exceptions #4

Merged
merged 2 commits into from
Nov 19, 2024

Conversation

LuigiBKL
Copy link
Collaborator

  • Limitation de la longueur des token à 3000 caractères pour l'api score/
  • légère refactorisation

@LuigiBKL LuigiBKL closed this Nov 19, 2024
@LuigiBKL LuigiBKL reopened this Nov 19, 2024
@Benvii
Copy link
Member

Benvii commented Nov 19, 2024

Hi @LuigiBKL limiting token size to 3000 characters will have impacts on the Orchestrator (Tock) we have a lot of documents chunks that represent more tokens, is there any infrastructure related restriction ? Maybe it should be configurable using environment variables. cc @assouktim

We can give you access to langfuse if you want to this the kind of chunk we are dealing with.

@killian-mahe
Copy link
Collaborator

Since the token limit has been removed and there are no comments on the Pull Request, I will merge it into the main branch.

@killian-mahe killian-mahe merged commit f422cb2 into main Nov 19, 2024
1 check passed
@killian-mahe killian-mahe deleted the fix/gestion_plus_fine_des_exceptions branch November 19, 2024 16:59
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.

4 participants