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

Add memory overlap check to meta_copy_ #108989

Closed
wants to merge 6 commits into from
Closed

Conversation

int3
Copy link
Contributor

@int3 int3 commented Sep 10, 2023

Stack from ghstack (oldest at bottom):

Fixes test_copy_many_to_one.

Fixes `test_copy_many_to_one`.

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 10, 2023

🔗 Helpful Links

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

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

✅ No Failures

As of commit 537ab59 with merge base 8ff0036 (image):
💚 Looks good so far! There are no failures yet. 💚

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

int3 added a commit that referenced this pull request Sep 10, 2023
Fixes `test_copy_many_to_one`.

ghstack-source-id: e7585cc9e737ff9970ff6c1dc76b62cd0dae0bb4
Pull Request resolved: #108989
@int3 int3 mentioned this pull request Sep 10, 2023
48 tasks
Fixes `test_copy_many_to_one`.

[ghstack-poisoned]
int3 added a commit that referenced this pull request Sep 11, 2023
Fixes `test_copy_many_to_one`.

ghstack-source-id: 76c0fa166dfeaff158e9815120fd7a94a70319ed
Pull Request resolved: #108989
@int3 int3 marked this pull request as ready for review September 11, 2023 15:07
@int3
Copy link
Contributor Author

int3 commented Sep 11, 2023

Test failures seem unrelated

Fixes `test_copy_many_to_one`.

[ghstack-poisoned]
int3 added a commit that referenced this pull request Sep 11, 2023
Fixes `test_copy_many_to_one`.

ghstack-source-id: 9f0cebaf4b09cabec91fc920a334a5ff9d2b8854
Pull Request resolved: #108989
@int3 int3 requested a review from eellison September 11, 2023 18:15
Copy link
Contributor

@eellison eellison left a comment

Choose a reason for hiding this comment

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

looks good -

torch/_meta_registrations.py Outdated Show resolved Hide resolved
Fixes `test_copy_many_to_one`.

[ghstack-poisoned]
int3 added a commit that referenced this pull request Sep 12, 2023
Fixes `test_copy_many_to_one`.

ghstack-source-id: 56260b0e276ee95ec80bc4f1e2085fa95ee2de5d
Pull Request resolved: #108989
Fixes `test_copy_many_to_one`.

[ghstack-poisoned]
int3 added a commit that referenced this pull request Sep 12, 2023
Fixes `test_copy_many_to_one`.

ghstack-source-id: ef6ff8e59276a10c708bc05ffdb13ff1fc3d8395
Pull Request resolved: #108989
Fixes `test_copy_many_to_one`.

[ghstack-poisoned]
int3 added a commit that referenced this pull request Sep 12, 2023
Fixes `test_copy_many_to_one`.

ghstack-source-id: 09eb7d17cbb88279c12bd62b34139d0af9804303
Pull Request resolved: #108989
@int3
Copy link
Contributor Author

int3 commented Sep 12, 2023

@pytorchbot merge

@pytorch-bot
Copy link

pytorch-bot bot commented Sep 12, 2023

This PR needs to be approved by an authorized maintainer before merge.

@int3 int3 added the topic: not user facing topic category label Sep 12, 2023
@int3
Copy link
Contributor Author

int3 commented Sep 12, 2023

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Sep 12, 2023
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@facebook-github-bot facebook-github-bot deleted the gh/int3/8/head branch September 16, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk Trigger trunk jobs on your pull request Merged topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants