-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add chaining of filters to select an IPA Provider #56
Labels
Comments
schnelle
added a commit
that referenced
this issue
Sep 27, 2024
Introduced `getSupportedLanguages` method in `ChatGPTIPAProvider` to retrieve supported languages and initialized the list in the constructor. Added `ProviderSelectionStrategyList` class to manage provider selection strategies. Enhanced `Language` class with equality operator. Updated `CMakeLists.txt` to include new files. Improved code readability with documentation comments.
schnelle
added a commit
that referenced
this issue
Sep 30, 2024
Introduce `LanguageMatchingProviderSelectionStrategy` to filter IPA providers based on language support. Add `getInputLanguages` method to `MultiModalInputs` for retrieving supported languages. Update CMakeLists.txt to include new files. Ensure new classes and methods are integrated and follow project standards.
schnelle
added a commit
that referenced
this issue
Oct 1, 2024
Refactored `ChatGPTIPAProvider.cpp` for better readability by reformatting the addition of the "Content-Type: application/json" header. Updated `main.cpp` to include new header files `LanguageMatchingProviderSelectionStrategy.h` and `ProviderSelectionStrategyList.h`. Modified the `main` function to use a `ProviderSelectionStrategyList` for more flexible provider selection, incorporating both `LanguageMatchingProviderSelectionStrategy` and `ModalityMatchingProviderSelectionStrategy`.
schnelle
added a commit
that referenced
this issue
Oct 7, 2024
The log message in the `ChatGPTIPAProvider::processInput` method has been updated to include the language of the text input. This enhancement provides more detailed context by logging the language along with the session ID, request ID, and request data.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, only one IPAProviderSelectionStrategy is possilbe.
In order to reuse and simplify development it is wishful to have multiple basic IPAProviderSelectionStrategies that can be chained together.
The text was updated successfully, but these errors were encountered: