Path: /knowledge/concepts/embedder/voyageai/overview
Hi, I am just getting familiar with Agno, awesome work!
By looking at the VoyageAIEmbedder implementation here:
https://github.com/agno-agi/agno/blob/main/libs/agno/agno/knowledge/embedder/voyageai.py
it looks like the "model" name should actually be provided in the "id" attribute of a VoyageAIEmbedder.
Since we are on the topic, please beware the current default "voyage-2" is getting old (current flagship model is "voyage-4").
Removing completely the default value for the (id/model) field may probably be the best option, this way:
- it would be up to the user to specify which model flavor to use
- it would prevent updates in the VoyageAIEmbedder definition from causing potential regressions when current users update the agno library and the default model would otherwise be updated implicilty.
Thanks!
Path: /knowledge/concepts/embedder/voyageai/overview
Hi, I am just getting familiar with Agno, awesome work!
By looking at the VoyageAIEmbedder implementation here:
https://github.com/agno-agi/agno/blob/main/libs/agno/agno/knowledge/embedder/voyageai.py
it looks like the "model" name should actually be provided in the "id" attribute of a VoyageAIEmbedder.
Since we are on the topic, please beware the current default "voyage-2" is getting old (current flagship model is "voyage-4").
Removing completely the default value for the (id/model) field may probably be the best option, this way:
Thanks!