-
Notifications
You must be signed in to change notification settings - Fork 578
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ask contribution to Tensorflow addons for general scope utils, loss, layers, ops #223
Comments
/cc @dynamicwebpaige If she is interested. |
See also the approved RFC https://github.com/tensorflow/community/blame/master/rfcs/20190802-model-garden-redesign.md#L60-L61 |
I will add in the target contributions repos the new keras-cv and keras-nlp. |
E.g. See tensorflow/models#8962 (review) |
@bhack still interested in this? |
In general this is a still valid topic but:
|
As we have just refreshed the model repo as model garden I would enforce the contributions policies of generale use (or already established in literature utils, losses, layers, ops to be contribute more systematically in Tensorflow/addons instead to be embedded or duplicated in the model repos.
E.g:
tensorflow/addons#361 (comment)
tensorflow/addons#1364 (comment)
tensorflow/addons#1366
tensorflow/addons#1226 (comment)
tensorflow/addons#1226 (comment)
/cc @ewilderj @facaiy @seanpmorgan
Policies to be enforced with a PR
https://github.com/tensorflow/models/blob/master/CONTRIBUTING.md
https://github.com/google-research/google-research/blob/master/CONTRIBUTING.md
https://github.com/google/automl/blob/master/CONTRIBUTING.md
The text was updated successfully, but these errors were encountered: