Skip to content

Conversation

@qlzh727
Copy link
Member

@qlzh727 qlzh727 commented Oct 5, 2022

What does this PR do?

Not sure why the unit test didn't catch it in #875. All the test check was passing.

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? Please add a link
    to it if that's the case.
  • Did you write any new necessary tests?
  • If this adds a new model, can you run a few training steps on TPU in Colab to ensure that no XLA incompatible OP are used?

Not sure why the unit test didn't catch it in keras-team#875. All the test check was passing.
@qlzh727
Copy link
Member Author

qlzh727 commented Oct 5, 2022

/gcbrun

Copy link
Contributor

@ianstenbit ianstenbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be worth digging into why our CI didn't catch this. From looking at the logs, it seems like the serialization test is run, so I'm not certain what happened.

@qlzh727 qlzh727 merged commit ec73b5a into keras-team:master Oct 5, 2022
ghost pushed a commit to y-vectorfield/keras-cv that referenced this pull request Nov 16, 2023
Not sure why the unit test didn't catch it in keras-team#875. All the test check was passing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants