dspy icon indicating copy to clipboard operation
dspy copied to clipboard

Published 20 hours ago • verified publisher icon stanfordnlp

fix react.py #276

Open dannyoo opened this issue 1 year ago • 3 comments

fix #276

dannyoo avatar Feb 07 '24 19:02 dannyoo

Hello, Any idea when this will be merged? Thank you!

younes-io avatar Feb 14 '24 11:02 younes-io

Thank you @dannyoo ! I just need this to be temporarily backward compatible.

Can we do: try/except, if there's .passages use that, else do this new behavior? I know that's slightly hacky but we may need it for a while. We can make it deprecated / issue a warning.

okhat avatar Feb 17 '24 05:02 okhat

Will merge otherwise after that!

okhat avatar Feb 17 '24 05:02 okhat

LGTM for the PR, it'd be great to take the changes from @smwitkowski .

insop avatar Feb 19 '24 00:02 insop