arnavsinghvi11
arnavsinghvi11
Thanks @mikeedjones . Could you run `ruff check . --fix-only` and push again? Ready to merge after that. To confirm, this change is more comprehensive than #918 and that PR...
Hi @BeatrixCohere , why is the `client_name=dspy` needed? I have not used Cohere myself, but curious where this client_name is getting used.
Hi @Su3h7aM , thanks for raising this PR! These changes are similar to #744 as you've mentioned, and in both cases, this breaks existing caches, so we can't merge the...
Hi @isbecker , thanks for the PR! Could you also add documentation for the CloudfareAI LM to provide context as done for the LMs in [our documentation here?](https://github.com/stanfordnlp/dspy/tree/main/docs/api/language_model_clients)? Also, could...
Hi @bllchmbrs , thanks for raising this. Activating DSPy Assertions/Suggestions actually requires wrapping the SectionWriter class with the backtracking/Retry logic. This will ensure that when the Suggestion is flagged with...
Hi @rpgoldman , thanks for the contributions to the documentation. These are much needed! The diagram is great. Could you remove the other image formats and add just the `png`...
> Would it be OK to retain the .dot file, since that is the source of all the other formats? Would it help to put a comment into the Markdown...
Hi @Su3h7aM , thanks for the PR! is it better to instead refactor into logging instead of removing entirely?
Thanks for the PR @mkotlarz ! From #887 , I'm curious if removing the specified finalized lines would correct the change instead of modifying the import? Ideally, we'd like to...
ah this is not rewriting langchain code, but rather the lines in [langchain.py](https://github.com/stanfordnlp/dspy/blob/d09d984ecaf17f7262294d50fe46fd8105fbf291/dspy/predict/langchain.py#L79) that may not be needed given the new `dspy.InputField` and `dspy.OutputField`. Does removing those lines resolve the...