Closed meditans closed 11 months ago
Yeah it's probably ready to merge. We just need to test it out with a recent vllm version.
Have you given it a shot?
I have written a quick version for myself that works with current vLLM, adapting the idea of that commit. I can post a PR if that helps! (Edit: done)
Just mentioning this here to avoid anyone being confused by error messages like:
while following the documentation. See https://github.com/stanfordnlp/dspy/pull/93