Matt

Results 203 comments of Matt

@D-roberts I think it's fine to swap those tests for just checking the actual argmax index rather than the `id2label` string value. Obviously the repository config doesn't actually have the...

LGTM now - I'm happy to merge as soon as you and @amyeroberts are!

Sorry for the delay! I still feel like we might need to refactor our model for what custom pipelines actually do, but in the meantime this seems okay to add....

Hi @kouroshHakha, just to speed things up a bit, can you give me a specific model ID and commit hash to use in the reproduction script that triggers this bug?

I'm still planning to investigate this when I get a chance too!

@ArthurZucker we still don't have the Keras 3 backend in place - I have a partial PR, but I'm not sure the team is ready for a fourth framework in...

Hi @a8nova, here's a [port of the Gemma modeling code](https://gist.github.com/Rocketknight1/77003f78147a9485a0f619e7202bb030)! Let me know if you need anything else. I did this port with Claude 3, and if you want to...

I just realized that I never told it to add things like `@keras_serializable` or `@unpack_inputs` but it knows the structure of `transformers` well enough that it did it anyway, lmao....

Hi @a8nova, my guess is there's some kind of version mismatch here between the `transformers` that's installed, and the `transformers` repo you're running the tests in. Try `pip install -e...