-
Notifications
You must be signed in to change notification settings - Fork 26.3k
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
Added type hints for Graphormer
pytorch version
#23073
Added type hints for Graphormer
pytorch version
#23073
Conversation
added type hints for graphormers pytorch , checked formating issues .
The documentation is not available anymore as the PR was closed or merged. |
This looks pretty good! Is there a reason to use |
@Rocketknight1 Hi 👋
if still changes are required i would be happy to make it🙂 |
Hi @dewasahu2003, I think in most cases we just annotate those types as |
@Rocketknight1 Hi 👋
|
Yep, I think replacing with LongTensor is slightly better, and does make the code a bit cleaner too. |
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.
This looks good to me now! Are you ready for me to merge it?
Sure |
Done. Thanks for the PR, we really appreciate it! |
* Added type hints for `Graphormer` pytorch version added type hints for graphormers pytorch , checked formating issues . * made the code less bloated
* Added type hints for `Graphormer` pytorch version added type hints for graphormers pytorch , checked formating issues . * made the code less bloated
* Added type hints for `Graphormer` pytorch version added type hints for graphormers pytorch , checked formating issues . * made the code less bloated
@Rocketknight1 👋
graphormer
pytorchif some checks on ci/cd do not, please do comment and correct