arnavsinghvi11
arnavsinghvi11
Closing for now to avoid breaking cache errors.
Thanks @stevenhoelscher for the PR! This looks good and maintains the cache correctly. Is the PR ready to merge or pending [the change you've mentioned here](https://github.com/stanfordnlp/dspy/pull/839#discussion_r1567678653)?
Hi @fsndzomga , just bumping the comments again as the PR is not yet mergeable. would love to merge this once those are resolved!
Thanks @fsndzomga ! Merging this, and feel free to open another PR with additional changes and documentation!
Thanks @isbecker !
Merged. thanks @BeatrixCohere !
Thanks @rpgoldman for this amazing PR on documentation. Left a small comment for you to give yourself credit for the PNG and should be ready to merge. (I left the...
Thanks @rpgoldman !
Hi @Su3h7aM , just following up on the comment above. Let me know if anything is unclear!
Thanks for circling back to this @rpgoldman @cdowellmdb ! Left a few comments back on the review and it should be good to merge once the last parts are resolved!...