Insop comments

Results 100 comments of


                                            Insop

Allow custom text key to WeaviateRM - resolve issue #359

With a cursory check, it looks great! Hi @CShorten, You could take this suggestion to *a separate PR.* I think it'd be great if you could add WeaviateRM description to...

Lower default `max_labeled_demos` in `BootstrapFewShot`

Sounds sensible to me. > ... like ofc... What is ofc?

fix: manually iterate kwargs['n'] in anyscale

@paxcema Nice. Could you also update `class Together(HFModel)` as well? it is the same code? If not I will follow up after this PR. Thank you.

fix: manually iterate kwargs['n'] in anyscale

Agree, I did not like the way it is now either since it is against `DRY`. If you have any suggestion, such as using a class that contains most common...

Create Google LM

Thank you so much @CShorten! It'd be great if you could update this markdown doc as well. `docs/language_models_client.md`

Add `metric_threshold` argument to BootstrapFewShot

I think this good idea for LM metric. Is there a way to add to add description as a docstring or related markdown file? Otherwise, LGTM.

Add CohereVectorizer for Embed 3

LGTM

Enable passing a non-default profile to dspy.Bedrock

LGTM

fix: update imports to allow global dspy.Google

One minor question, and looks good to me. Thank you,

feat(retrieve): add support for faiss retriever, update setup.md, docs

Thank you. You could just merge/rebase the branch from PR #439 instead.