Default compilation other classifiers #714
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
chenmoneygithub
left a comment
chenmoneygithub
left a comment
There was a problem hiding this comment.
Thank you for the PR!
PR looks good! there is a formatting issue caused by black version change. Please see the comment above.
| mask_positions, | ||
| mask_ids, | ||
| ) = tf_text.mask_language_model( | ||
| (token_ids, mask_positions, mask_ids,) = tf_text.mask_language_model( |
There was a problem hiding this comment.
This formatting is due to a black version change. Could you upgrade black and rerun the formatting? Thanks!
jbischof
left a comment
jbischof
left a comment
There was a problem hiding this comment.
Looks amazing. Welcome to KerasNLP @Plutone11011!
4 participants
Resolve #709
Second commit is due to reformatting