search

acsresearch / interlab

MIT License
17 stars 3 forks source link

Bug in AnthropicEngine.query and AnthropicEngine.aquery #21

Closed jas-ho closed 11 months ago

jas-ho commented 11 months ago

see #20

note: unit tests currently rely on API access and are skipped if corresponding environment variables for API keys are not present. there may be a better solution for this

gavento commented 11 months ago

Thanks for the fix and for the (very nice) test!

I think this happened in 33dcd22, when switching Anthropic from 0.2 to 0.3. (We are mostly using langchain wrappers lately, and may only keep ours around for any future special functionality.)

Closes #20