Support null captures (Fix #826)

guidance-ai / guidance

A guidance language for controlling large language models.

MIT License

18.14k stars 1.01k forks source link

Support null captures (Fix #826) #834

Closed slundberg closed 1 month ago

slundberg commented 1 month ago

This fixes an issue where selecting or generating empty strings does not get recorded as a named capture (as noted in #826).

slundberg commented 1 month ago

@riedgar-ms @Harsha-Nori it seems like hfllama is unhappy for one of the tests. I might just ignore and merge for now, but any thoughts I what's up? Thanks!

riedgar-ms commented 1 month ago

Not sure exactly what's up with the LLamaCpp test. It has been a lot more temperamental over the past couple of days. So long as everything else is green, I wouldn't block on merging.

Harsha-Nori commented 1 month ago

Test failures appear unrelated. LGTM.