search

keras-team / keras-hub

Pretrained model hub for Keras 3
Apache License 2.0
804 stars 243 forks source link

Update PaliGemma to remove `include_rescaling` arg #1917

Closed divyashreepathihalli closed 1 month ago

SamanehSaadat commented 1 month ago

@divyashreepathihalli Could you share a little bit of context about this change?

divyashreepathihalli commented 1 month ago

@divyashreepathihalli Could you share a little bit of context about this change?

Hey @SamanehSaadat. for Vision models we have added a preprocessor flow that handles resizing and rescaling. So I am removing this arg from PaliGemma backbone. This will be a breaking change but it is necessary to keep our models consistent.

SamanehSaadat commented 1 month ago

@divyashreepathihalli Could you share a little bit of context about this change?

Hey @SamanehSaadat. for Vision models we have added a preprocessor flow that handles resizing and rescaling. So I am removing this arg from PaliGemma backbone. This will be a breaking change but it is necessary to keep our models consistent.

This PR https://github.com/keras-team/keras-hub/pull/1927, right?