Closed SrNandu closed 3 weeks ago
The LLamaSharpEmbeddings.cs
file was updated to reorder import statements and introduce a new property EmbeddingMode
in the LLamaSharpConfiguration
class. This property is now initialized in the FromPath
method and utilized in the constructor to prevent an exception.
File Path | Change Summary |
---|---|
src/Providers/LLamaSharp/src/LLamaSharpEmbeddings.cs |
Reordered using statements, added EmbeddingMode to LLamaSharpConfiguration , initialized it in FromPath method, and utilized it in constructor. |
The changes are too straightforward and varied to be encapsulated in a single sequence diagram. Therefore, no sequence diagram is provided.
Objective | Addressed | Explanation |
---|---|---|
EmbeddingMode must be true when creating LLamaSharpEmbeddings (#343) |
✅ |
In fields of code, the rabbit roves, EmbeddingMode where logic hoves. With imports set and paths defined, The Llama's course is now refined. Debugging trails we deftly close, In binary poems, our joy arose.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Thank you very much for your contribution. Please write me your name in discord if you want and I will add you to the list of contrubutors and dev channel discussing development and immediate plans You can join using https://discord.gg/Ca2xhfBf3v
Fixes tryAGI/LangChain#343
Fixed by passing the EmbeddingMode configuration from LlamaSharpConfiguration to the ModelParams. Also this propery is setted true in the FromPath() static method, otherwise this method will crash too.
Summary by CodeRabbit
EmbeddingMode
property to enhance the configuration of embeddings.EmbeddingMode
property, allowing users to set embedding modes more flexibly.