Skip to content
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

Adjust maintainers list #23693

Closed
wants to merge 2 commits into from
Closed

Conversation

dzhulgakov
Copy link
Collaborator

Adds new people and reorders sections to make more sense

@dzhulgakov dzhulgakov requested a review from soumith August 1, 2019 23:01
@pytorchbot pytorchbot added the module: docs Related to our documentation, both in docs/ and docblocks label Aug 1, 2019
@soumith
Copy link
Member

soumith commented Aug 1, 2019

@xuhdev let us know if you're comforable being listed for build system
@vishwakftw let us know if you are comfortable with your name also listed under linear algebra

@xuhdev
Copy link
Collaborator

xuhdev commented Aug 1, 2019

@soumith Yes, please. Thanks!


- Edward Yang (`ezyang <https://github.com/ezyang>`__)
- Alban Desmaison (`alband <https://github.com/alband>`__)
- (sunsetting) Adam Paszke (`apaszke <https://github.com/apaszke>`__)
Copy link
Contributor

Choose a reason for hiding this comment

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

FWIW I'm reading every single autograd PR, even if I don't comment on it, so I wouldn't agree with that 😬

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, sure, it was @soumith's suggestion :-P

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @soumith 😊

- Ailing Zhang (`ailzhang <https://github.com/ailzhang>`__)
- Gregory Chanan (`gchanan <https://github.com/gchanan>`__)
- Davide Libenzi (`dlibenzi <https://github.com/dlibenzi>`__)
- Alex Suhan (`asuhan <https://github.com/asuhan>`__)
Copy link
Contributor

Choose a reason for hiding this comment

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

IIUC Alex is no longer working on the XLA bridge

@vishwakftw
Copy link
Contributor

vishwakftw commented Aug 2, 2019

@vishwakftw let us know if you are comfortable with your name also listed under linear algebra

Yes, very much. Thank you!

Copy link
Member

@soumith soumith left a comment

Choose a reason for hiding this comment

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

go go go!

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@dzhulgakov is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@dzhulgakov merged this pull request in acc5ced.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged module: docs Related to our documentation, both in docs/ and docblocks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants