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

Adds MetalRetriever, HugginfaceEndpoint and other bugfixes #837

Merged
merged 18 commits into from
Aug 29, 2023
Merged

Conversation

ogabrielluiz
Copy link
Contributor

No description provided.

ogabrielluiz and others added 14 commits August 28, 2023 07:32
…ata using the Metal API

✨ feat(__init__.py): add empty __init__.py file to the retrievers directory to make it a package
…o langchain/schema to keep codebase up to date

🔀 merge(MetalRetriever.py): merge changes from metal_sdk/metal to langchain/schema to keep codebase up to date
🐛 fix(MetalRetriever.py): change BaseRetriever import from langchain.schema.retriever to langchain.schema to match updated codebase
✨ feat(MetalRetriever.py): add error handling to MetalRetriever build method to raise ValueError if connection to Metal API fails
… metal_sdk.metal module to resolve type checking issue
…alue conversion from string to array format

📝 WHY: The InputListComponent expects an array format for the value prop, but sometimes it receives a string instead. To prevent errors and ensure smooth operation, we need to handle the conversion from a string to an array with the string as its element. This refactor improves code readability and clarifies the purpose of the value conversion logic.
… from string to array format

🔧 chore(inputListComponent): add recursive character text splitter to handle conversion from string to array format
…se parameter to improve code readability and maintainability

🔀 refactor(OpenAIConversationalAgent.py): make openai_api_base parameter optional with a default value of None for better flexibility and backward compatibility
Added component to use an LLM model from Hugging Face Inference API.
@ogabrielluiz ogabrielluiz added the Release Label to be set only on release PR label Aug 29, 2023
@render
Copy link

render bot commented Aug 29, 2023

@ogabrielluiz ogabrielluiz merged commit d5faee5 into main Aug 29, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Label to be set only on release PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants