arnavsinghvi11

Results 149 comments of arnavsinghvi11

Hi @aazizisoufiane , a PR would definitely be welcome! Just curious though, which AWS model were you using for this? I'm assuming Anthropic since I only see that AWS LM...

Hi @kalpit-konverge , do you have a specific use-case? We used to support 3.8 but have since upgraded to 3.9. We can lower it if needed but will require some...

@lambdaofgod you can set the LM per step through the context manager and dynamically change the respective LM's tokens as needed: ``` with dspy.context(lm=dspy.OpenAI(...., max_tokens= x)): .... with dspy.context(lm=dspy.OpenAI(...., max_tokens=...

@lambdaofgod would `dspy.settings.config['lm'].kwargs['max_tokens'] = ..` cover the abstraction here? you wouldn't have to expose the defined lm and can still wrap modules using `with dspy.context(lm = lm)` ``` lm =...

@lambdaofgod They are quite similar, just that the reference I provided is already built in with DSPy and wouldn't require any additional changes :).

Makes sense @lambdaofgod . Feel free to push a PR that can better handle this behavior in `dspy.context` without impacting existing settings!

Thanks a lot for these additions @Athe-kunal ! Really appreciate the tutorial notebook! I've left some comments for the PR to mainly address some code cleanup changes. It would also...

Hi @Athe-kunal , just left some follow up comments. it seems that there are still some leftover comments to address. feel free to discuss them more directly here as needed.

Hi @Athe-kunal , thanks again. I'd love to merge this PR but unfortunately, it breaks some existing caches we have in the repository, particularly the intro.ipynb. For more clarity, the...

Thanks @drawal1 for the PR! left a few comments. The changes to Ensemble look correct to me, but I think the ones for RandomSearch are not needed? or can be...