Uses Collection in transformers.image_transforms.normalize

[CalOmnie]

What does this PR do?

Followup to this PR
Which introduced a bug, preventing numpy arrays to be used as std and mean parameters.
This PR uses instead the Collection type, which correctly check for the prence of both __len__ and __getitem__ attributes but not __reversed__, succesfully accepting numpy arrays

This PR also updates one of the tests to pass in numpy arrays to confirm this new type does indeed function as intended.

Before submitting

• This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
• Did you read the contributor guideline,
  Pull Request section?
• Was this discussed/approved via a Github issue or the forum? Please add a link
  to it if that's the case.
• Did you make sure to update the documentation with your changes? Here are the
  documentation guidelines, and
  here are tips on formatting docstrings.
• Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@Rocketknight1
@tomasruizt

[Rocketknight1]

Hi @CalOmnie, yes, this seems like a good typing update! There's one issue, though: typing.Collection is marked as a deprecated alias - can we use collections.abc.Collection instead?

[CalOmnie]

Hey @Rocketknight1 fair enough, made the change.

[Rocketknight1]

Yes, LGTM!

cc @ArthurZucker @Cyrilvallez for core maintainer review, and note this comment in the previous PR. isinstance(mean, Sequence) actually introduced a bug because np.ndarray is not a subclass of Sequence, so this is a bugfix and not just a type hint update!

[Cyrilvallez]

LGTM, thanks for the fix!! 🤗

[tomasruizt]

Thanks for the quick fix @CalOmnie