arnavsinghvi11
arnavsinghvi11
Hi @aazizisoufiane @JamesScharf , does #720 handle this? wondering if we can close this PR and make the changes to Bedrock as @JamesScharf suggests?
@drawal1 it seems like we could incorporate some of the AWS model error handling (particularly backoff) from this PR. feel free to suggest any comments on how to merge with...
> @arnavsinghvi11 @aazizisoufiane Both the files in this PR are now obsolete. > > Based on a review of the backoff handler, the changes belong in the aws_providers.py file. Specifically,...
Hi @aazizisoufiane , can you run `ruff check . --fix-only` and push again? should be good to merge after this!
Hi @fsndzomga , thanks for the addition! We want to keep all external imports/dependencies local to the file requirements.
@fsndzomga @axelpey Thanks for the discussion on the limitation with ChatCompletion models. This is indeed something we've seen come across with running chat models with DSPy. There is some more...
Tagging the corresponding [README](https://github.com/stanfordnlp/dspy/tree/main/docs#readme) for updating docs as well!
tagging @isaacbmiller for the failed checks. seems like a simple docs modification but somehow failing tests?
Thanks @ovshake ! updated in docs.
@mwildehahn @isaacbmiller tests still failing