Skip to content

[primTorch] Add repeat and unfold_copy references#81374

Closed
rdspring1 wants to merge 20 commits intopytorch:masterfrom
rdspring1:ref_repeat
Closed

[primTorch] Add repeat and unfold_copy references#81374
rdspring1 wants to merge 20 commits intopytorch:masterfrom
rdspring1:ref_repeat

Conversation

@rdspring1
Copy link
Contributor

@rdspring1 rdspring1 commented Jul 12, 2022

Add References:

  • repeat
  • unfold
  • expand_as

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jul 12, 2022

🔗 Helpful links

✅ No Failures (0 Pending)

As of commit b79b179 (more details on the Dr. CI page):

Expand to see more

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


This comment was automatically generated by Dr. CI (expand for details).

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

Click here to manually regenerate this comment.

@rdspring1 rdspring1 marked this pull request as ready for review July 13, 2022 23:55
@ngimel ngimel added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jul 14, 2022
rdspring1 added a commit to rdspring1/pytorch that referenced this pull request Jul 14, 2022
@soulitzer
Copy link
Contributor

Hey @rdspring1 do you also plan to implement the ref for repeat_interleave? I started looking into it, but realized later that you already picked up the packet.

@rdspring1
Copy link
Contributor Author

I've looked into repeat_interleave, but it isn't a high priority reference for nvfuser. I wasn't planning on implementing it in the near-term.

@IvanYashchuk
Copy link
Collaborator

@ngimel could you please take a look at this PR?

@pytorch-bot
Copy link

pytorch-bot bot commented Sep 8, 2022

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/81374

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 43df14a:
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

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 - see small inline comments and this may require a rebase

@rdspring1
Copy link
Contributor Author

@pytorchbot merge -g

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: Approval needed from one of the following (Rule 'superuser'):
davides, highker, xta0, bilalsal, houseroad, ...

Details for Dev Infra team Raised by workflow job

@rdspring1
Copy link
Contributor Author

@ngimel Could you look at this?

@ngimel
Copy link
Collaborator

ngimel commented Sep 12, 2022

@pytorchbot merge -g

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a merge job. Check the current status here.
The merge job was triggered with the green (-g) flag. This means that your change will be merged once all checks on your PR have passed (ETA: 0-4 Hours). If this is not the intended behavior, feel free to use some of the other merge options in the wiki.
Please reach out to the PyTorch DevX Team with feedback or questions!

@github-actions
Copy link
Contributor

Hey @rdspring1.
You've committed this PR, but it does not have both a 'release notes: ...' and 'topics: ...' label. Please add one of each to the PR. The 'release notes: ...' label should represent the part of PyTorch that this PR changes (fx, autograd, distributed, etc) and the 'topics: ...' label should represent the kind of PR it is (not user facing, new feature, bug fix, perf improvement, etc). The list of valid labels can be found here for the 'release notes: ...' and here for the 'topics: ...'.
For changes that are 'topic: not user facing' there is no need for a release notes label.

mehtanirav pushed a commit that referenced this pull request Oct 4, 2022
Add References:

- repeat
- unfold
- expand_as
Pull Request resolved: #81374
Approved by: https://github.com/mruberry, https://github.com/ngimel
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged module: primTorch open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants