Alexandre E. Souza
Alexandre E. Souza
An initial idea ```go func removeEmptyKeys(originalMap map[string]Definition) map[string]Definition { filteredMap := make(map[string]Definition) for key, value := range originalMap { if key != "" { if value.Properties != nil { value.Properties...
It would be beneficial to have tests for methods other than GetRelevantDocuments. This approach would make the implementation clearer.
> From what I can tell, to use things like Tools, the correct approach is to call GenerateContent on a model - but all of the chains use Call(). >...
> `ollama pull model` is too slow and the experience is not very good with test container, Did you think it was slow?
> Also I notice that the build examples is failing, this is due to /memory/zep import not being available yet. for example don't need use go.mod , can import only...
@tmc could review it?
@paul-paliychuk why update to draft?
> I'm a little skeptical about adding APITypeNvidia as I feel a different BaseURL should be sufficient in most cases. I thought about this possibility, but if you just use...
> I just landed #733 and added an example -- can you rebase this? > > I'm a little skeptical about adding APITypeNvidia as I feel a different BaseURL should...
> Yeah still not open to doing this in such a vendor specific manner, I'd be open to a WIth* Option that adds this but I don't think we should...