-
Notifications
You must be signed in to change notification settings - Fork 1
TTYG-166 Configurable LLM #50
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
Merged
Changes from all commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
269d4fe
Configure LLM. TODO: README, system_tests
pgan002 a1c34ba
Rename config_data/ config files
d47275b
Update system tests to use new config
29d3a75
Rename tests-with-openai/ to tests-with-llm
6ddb063
Update README
af35961
README clarifications
4fbf79d
Complete config spec. Example LLM-only config
754d6ce
Clean up variable name llm_config_
63b77e0
Remove debugging print() from run_evaluation()
d2fbd74
llm.py, README: temperature <=0.0, >=2.0
4bc6249
Pass undeclared LLM config fields to completions()
08026cf
Fix test_retrieval_answer incl removing n/a cases
815164c
Fix test_llm after rebase
1ee0efe
Fix llm.call(): do not validate name after rebase
68689a0
Minor
8b25993
More tests fixes after rebase
59c8712
Fix system test answer_relevance
c9ebe34
Pass only generation config when possible
3f325a8
Pass Ragas llm and embedding clients
7d02858
Fix test_llm.py: move cases to tests_with_llm
d77fb64
README Update LLM config explanation, examples
43296ef
README: fix link to custom evaluation section
9d165f4
Rename class import
7b43d57
Fix method name litellm.completion(); deprecated
2284cc3
Declare parameter types for ragas llm and embedder
4041818
Rename `ragas_embeddings` to `ragas_embedder`
51c31e5
Revert unintended, irrelevant changes
f9f8ebd
test_custom_evaluation: double-quote (") -> '
f996d96
aggregation.py: move imports to top of file
9bcca87
answer_correctness CLI: limit temperature values
6671940
evaluation.py: positional arguments throughout
8eb210e
evaluation.Config method return types
e07c97e
README: fix links to sections
b363edf
class steps.retrieval_*.Evaluator to store llm
e7f88be
class answer_relevance.Evaluator
6f209fa
llm.py: dynamyc LLM client using litellm
bca113b
llm.create_llm_and_embedder() quoted return type
7e0edf9
Unused import
9673619
steps.evaluation.evaluate_steps() quoted return type
735afba
Pydantic
746fa65
dependencies: pydantic, litellm
56b9a2d
Fix LLM client and embeddings creation
9297094
Instantiate Ragas scorers in constructors
5623018
answer_correctness temperature= [0, 2]; def parse_args()
37c8e4c
llm.py: fix indentation
d69c0a7
Unused print()
b61f420
Consistently use " instead of '
f414226
Fix spacing
3949bc2
Use singleton evaluator classes
27507be
Redundant variable init
c4e7702
Fix litellm call. +system-tests/answer_correctness
1dfa970
Fix system-tests/answer_correctness: sync generate()
7c27d0b
Replace llm.generate() by a call to ragas_llm's .generate()
560829b
README optional config keys base_url, api_key
a7638a3
Fix tests-with-llm
435cda1
Add system_tests/answer_correctness_azure.py
8292ad2
Add system-tests/answer_relevance_azure.py
ee28d04
Split create_llm_and_embedder() into two functions
41befd1
README: example config with different api_key
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.