[Fix] Spelling error in "reogranize" and [Feat] Minor improvements to Encoder and CoStormRunner
#321
Conversation
This file contains 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
toml strings need to be in quotes
This flag is necessary when running the first time so IMO should be included in the readme example
Update README.md — fix secrets.toml syntax
Update README.md — aads --do-research flag to example
Update README.md — fixes You.com API key env variable
[fix typo] README.md
folloing -> following
Update citation_quality.py
adding tqdm lib
-using list comprehensions and more descriptive variable names. -using a case-insensitive check for keywords in section identification. -Added progress tracking using tqdm -renamed variables
Small improvements
…patch-add-prometheus-eval-prompt Create eval_prometheus_no_ref.prompt
…patch-modify-eval-path update evaluation data path
…ebase-sync Refactored codebase sync
auto-sync-2024-26-04-04-11-33
[New RM] Add AzureAISearch
valid_url_to_snippets.get(url, {}) get none value.
call strip function will be fix it. just like url.strip("\'")
valid_url_to_snippets.get(url, {}) get none value.
call strip function will be fix it. just like url.strip("\'").
format code and commit.
Fix SerperRM bug.
resolve issue raised in stanford-oval#243
update readme to include description of `ENCODER_API_TYPE` which specified which encoder to use in co-storm
…-patch-readme Update README.md
…zuremodel Fix `AzureOpenAIModel` to match the latest Azure API.
|
Fixes #327 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR has a two small improvements and fixes a spelling error.
Spelling error
reogranizetoreorganizein both thecollaborative_storm/engine.pyanddataclass.pyfiles.Optional Encoder Injection in CoStormRunner
CoStormRunnerinitialization to make encoder injection optional. If no encoder is provided, a defaultEncoderinstance is automatically created (matching current behavior).Support for Extra Headers in Encoder Initialization
extra_headersduring the initialization of theEncoderclass. This allows for additional header configurations when using azure embedding models.