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

[DO NOT MERGE] Implemented double llm call to significantly reduce the number of false positives #212

Open
wants to merge 2 commits into
base: Dev
Choose a base branch
from

Conversation

smrchanda877
Copy link
Collaborator

Before testing:

  • Create another api key and put in your .env file as MISTRAL_SECOND_API_KEY or just use the key below
    image
  • I noticed the false positives have reduced significantly and the response time is still pretty fast ~ 3 seconds

@smrchanda877
Copy link
Collaborator Author

@ACraig7 @Jagadeesh-ajjada @itsnotmik If we want to implement this method, then, we need to change the readme to pull from dockerhub instead of github.
Also, have two api keys from different accounts in the github secrets and fetch the api keys from there

@smrchanda877
Copy link
Collaborator Author

Test these changes first. If this is worth implementing, then someone can implement the docker related changes

@smrchanda877 smrchanda877 changed the base branch from main to Dev November 26, 2024 23:06
@smrchanda877
Copy link
Collaborator Author

changed from main to Dev as the latest code is in the Dev

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