Conversation
movchan74
left a comment
There was a problem hiding this comment.
Some changes are required. Don't forget to run Ruff. Also, merge the main into the branch.
|
Idefics 2 model card says that it doesn't work with some versions of transformers library.
See https://huggingface.co/HuggingFaceM4/idefics2-8b I suggest we bump the version of transformers library in pyproject. |
Bumped to 4.41.2 |
| return str(obj) | ||
| if isinstance(obj, type): | ||
| return str(type) | ||
| if isinstance(obj, np.ndarray): |
There was a problem hiding this comment.
ndarray should be handled by OPT_SERIALIZE_NUMPY in orjson. But if you use jsonify then it will not handle numpy. Can you check and see if we can unify jsonify and orjson_serializer and use orjson for everything?
Co-authored-by: Aleksandr Movchan <aleksandr@mobius.ml>
Summary:
This update added Idefics2 deployment into the available deployments
Related issues: #145