arnavsinghvi11

Results 259 comments of arnavsinghvi11

@drawal1 - could we potentially isolate that patch and push it within this PR? Seems like #843 is more involved and needs some conflicts to be resolved before merging

Thanks @usamajamil43 ! LGTM. Unfortunately, I cannot test the caches for this myself, but could you verify that with the existing behavior before pushing? Also, the PR has a few...

Hi @usamajamil43 , could you run `ruff check . --fix-only` and commit again? Ready to merge after that!

Hi @usamajamil43 , can you rebase to the latest version on main? should fix the failing ruff test and ready to merge.

Hi @usamajamil43 , I believe if you rebase to the latest commit on the main branch and then run ruff, this should fix the existing error. I tried doing this...

Thanks @usamajamil43 and @ahsansaeed878 for your patience with the PR! merged.

Hi @drawal1 , thanks for the changes. can you also revert the changes to [BootstrapWithRandomSearch](https://github.com/stanfordnlp/dspy/pull/843/files#diff-cfdd85de1987474649224c297047325dc2f0b8b890f7a45810556be440ecf84a) since those changes are not relevant to patching #858 .

Resolved with latest updates!