OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

Remove pydantic warnings

Open SmartManoj opened this issue 1 year ago • 1 comments

https://github.com/BerriAI/litellm/pull/2893

SmartManoj avatar Apr 08 '24 12:04 SmartManoj

Oh thanks! This is good news. LiteLLM is very quick with new patches, they released several within days lately.

I guess we can wait a little bit, rather than hardcode a branch. The warning is annoying though.

@dorbanianas WDYT?

enyst avatar Apr 08 '24 13:04 enyst

@enyst Yeah agree, the warning is too annoying, but as much as it doesn't affect the base execution of our project so we can wait for the update 👌

anasdorbani avatar Apr 08 '24 14:04 anasdorbani

👍 I agree on waiting. If it takes more than a day or two we can poke them to release

rbren avatar Apr 08 '24 14:04 rbren

Thanks for bringing it to our attention @SmartManoj! But let's close this one and just wait for the upstream to flow in

rbren avatar Apr 08 '24 14:04 rbren

I guess we can wait a little bit, rather than hardcode a branch. The warning is annoying though.

Merging it now. Sorry for the warning, do you have suggestions on how we can avoid these types of warnings from re-occuring ?

ishaan-jaff avatar Apr 09 '24 15:04 ishaan-jaff

@ishaan-jaff you can simply avoid using the model_ name for the pydantic class attributes more info here also can you pls update to the latest version so the issue would stop it's really annoying 😅

anasdorbani avatar Apr 09 '24 16:04 anasdorbani

@dorbanianas tracking this here: https://github.com/BerriAI/litellm/issues/2915

on our side we'll add testing to ensure that this warning will not re-appear

can you pls update to the latest version so the issue would stop it's really annoying

just queued a new release

ishaan-jaff avatar Apr 09 '24 17:04 ishaan-jaff