-
Notifications
You must be signed in to change notification settings - Fork 580
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
Updated charter for TFX Addons, updated core members #429
base: master
Are you sure you want to change the base?
Updated charter for TFX Addons, updated core members #429
Conversation
- SIG members: | ||
- <add yourself here to participate in SIG meetings & discussion> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: may be good to just point to Mailing list directly instead
|
||
We have multiple channels for participation, and publicly archive discussions in our user group mailing list: | ||
We have multiple channels for participation, and publicly archive discussions in our user group mailing list: | ||
- [email protected] – our general mailing list that all are welcome to join (archive group: https://groups.google.com/a/tensorflow.org/g/tfx) | ||
- [email protected] – Google group for core team (to be created) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can remove this and instead just reference the sig tfx-addons google group. No need to keep a separate group for core.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, we haven't been using that much. It does still exist, and we can use it, but it probably doesn't need to be mentioned here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Updated charter to reflect the new core team members.
Update is related to this change to the tfx-addons repo: tensorflow/tfx-addons#194