search

AbanteAI / spice

accelerant
Apache License 2.0
4 stars 2 forks source link

Fixes #102: don't choke on Azure metadata #104

Closed laszlovandenhoek closed 3 months ago

laszlovandenhoek commented 3 months ago

This is a fix for #102, similar to https://github.com/AbanteAI/mentat/pull/430.

biobootloader commented 3 months ago

Thanks @laszlovandenhoek!

I think this will break anthropic, which doesn't use choices. Perhaps the filtering should be done in the WrappedOpenAIClient.process_chunk? Or maybe even better you could make a process_chunk for WrappedAzureClient?

laszlovandenhoek commented 3 months ago

@biobootloader sorry, I got sloppy. I have opened a new PR: https://github.com/AbanteAI/spice/pull/105