arnavsinghvi11

Results 259 comments of arnavsinghvi11

@owen-deepskill Could you try using dspy from source instead of pypi? Seems like past [errors](https://github.com/stanfordnlp/dspy/issues/504) related to this are fixed through that but if not, we can take a closer...

Hi @arunpatro, I believe ``` class AssertReason(BaseModel): assertion: str = Field() reason: str = Field() answer: Literal["A", "B", "C", "D"] = Field() ``` should not have the type declarations here...

Thanks @cta2106 @aazizisoufiane! Should we merge both #680 and #720 in that case? the changes make sense to me but I just want to confirm there are no behavior conflicts...

@cta2106 looks good to merge after you run `ruff check . --fix-only` and push!

@drawal1 is this PR good to close as well with #795 merged?

Hi @drawal1 , could you share your program and stack trace? I suspect this is because `dspy.Ensemble` returns a list of DSPy programs and that's how they got saved in...

Hi @minfawang , thanks for checking up on this. the lm is actually used as the line `if self.lm is None` defaults to using the [`dspy.settings.lm` ](https://github.com/stanfordnlp/dspy/blob/dba1285fb8acdfd0154a1e7fdc0448e3eadd7dab/dsp/primitives/predict.py#L62)but internally when `self.lm`...

hmm I'm still a bit confused. Could you share an example perhaps that illustrates this issue? To my understanding, the current logic ensures the correct LM is loaded in, and...

@minfawang This will actually work using the DSPy context manager. Feel free to reference [this documentation on using multiple LMs together](https://dspy-docs.vercel.app/docs/building-blocks/language_models#using-multiple-lms-at-once). If you run `with dspy.context(lm= lm2):` instead of specifying...

I think the change you've proposed (1) is making more sense to me now because we don't want to ignore the lm kwarg if passed in, but I'm realizing `self.lm`...