KEMBAR78
add support for constant by praneethratna · Pull Request #60166 · pytorch/pytorch · GitHub
Skip to content

Conversation

@praneethratna
Copy link
Contributor

@praneethratna praneethratna commented Jun 17, 2021

Fixes #58739 Add support for constants according to python array API stipulation.

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jun 17, 2021

💊 CI failures summary and remediations

As of commit 9a744f9 (more details on the Dr. CI page and at hud.pytorch.org/pr/60166):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@praneethratna
Copy link
Contributor Author

@pmeier sorry as I have made a mess with commits , so have to delete the previous branch, I`ve updated here again

Copy link
Collaborator

@pmeier pmeier left a comment

Choose a reason for hiding this comment

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

sorry as I have made a mess with commits , so have to delete the previous branch, I`ve updated here again

Happens to the best of us 😉 You accidentally included updates to the ideep submodule in the branch. Here is how you get them out again:

$ cd $PYTORCH/third_party/ideep 
$ git checkout f9468ff1a3d601b509ebe2c17d2ed0a58dffacee
$ git commit -am "revert submodule update"
$ git push

If you run git submodule update after doing a git pull you won't have this problem in the future.

@pmeier
Copy link
Collaborator

pmeier commented Jun 17, 2021

For reference, the old PR is #59910.

@praneethratna
Copy link
Contributor Author

@pmeier I have tried your steps but i am getting a permission denied while trying to push could you help me figure it out?

@pmeier
Copy link
Collaborator

pmeier commented Jun 17, 2021

You need to perform these steps while on your branch. "Permission denied" most likely means that your trying to push to branch that isn't yours. Probably master.

@praneethratna
Copy link
Contributor Author

I have tried these steps on my branch itself but still it shows "Permission Denied"

@pmeier
Copy link
Collaborator

pmeier commented Jun 17, 2021

Ahh sorry, do a cd $PYTORCH before the commit.

@praneethratna praneethratna requested a review from pmeier June 17, 2021 11:12
@pmeier
Copy link
Collaborator

pmeier commented Jun 17, 2021

The commit is still there. Is it ok for you if I fix the branch so we can land this PR?

@praneethratna
Copy link
Contributor Author

yes absolutely not an issue 😃

Copy link
Collaborator

@pmeier pmeier left a comment

Choose a reason for hiding this comment

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

Should be good to go now. Thanks @praneethratna!

@pmeier pmeier requested a review from mruberry June 17, 2021 14:51
Copy link
Collaborator

@mruberry mruberry left a comment

Choose a reason for hiding this comment

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

Cool, thank you @praneethratna!

@facebook-github-bot
Copy link
Contributor

@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@mruberry merged this pull request in 9b30fb8.

@praneethratna praneethratna deleted the issue#58739 branch September 1, 2021 13:29
mikaylagawarecki

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Python Array API: support constants

6 participants