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 activation functions (ReLU and SiLU for now) for structured sparse linear operator #101339

Closed

Conversation

alexsamardzic
Copy link
Collaborator

@alexsamardzic alexsamardzic commented May 13, 2023

@pytorch-bot
Copy link

pytorch-bot bot commented May 13, 2023

🔗 Helpful Links

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

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

✅ 3 Unrelated Failures

As of commit 4dd1e44:

UNSTABLE - The following jobs failed but were likely due to flakiness present on trunk and has been marked as unstable:

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

alexsamardzic added a commit that referenced this pull request May 13, 2023
…e linear operator

ghstack-source-id: dbccf81feeae952f008d29b90ff12b3e2fa617ca
Pull Request resolved: #101339
@alexsamardzic alexsamardzic added module: sparse Related to torch.sparse topic: not user facing topic category labels May 13, 2023
…tured sparse linear operator"

cc nikitaved pearu cpuhrsch amjames bhosmer

[ghstack-poisoned]
alexsamardzic added a commit that referenced this pull request May 13, 2023
…e linear operator

ghstack-source-id: 1114550774c7f46e7b204f0137ab7599bfec1526
Pull Request resolved: #101339
@cpuhrsch
Copy link
Contributor

Overall the changes look reasonable. Can you check how much this increases our binary size? You can look at the size of libtorch_cuda.so. I'm worried we'll start adding a lot of size.

@alexsamardzic
Copy link
Collaborator Author

It doesn't seem too much of a difference: without my PRs, the size of libtorch_cuda.so is 152.51MB, and with them it's 153.15 MB.

@cpuhrsch
Copy link
Contributor

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label May 16, 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

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: PR #100881 has not been reviewed yet

Details for Dev Infra team Raised by workflow job

Failing merge rule: Core Maintainers

@amjames
Copy link
Collaborator

amjames commented May 17, 2023

@pytorchbot merge

@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

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: This PR is too stale; the last push date was more than 3 days ago. Please rebase and try again. You can rebase and merge by leaving the following comment on this PR:
@pytorchbot merge -r
Or just rebase by leaving @pytorchbot rebase comment

Details for Dev Infra team Raised by workflow job

@cpuhrsch
Copy link
Contributor

@pytorchbot merge -r

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

…tured sparse linear operator"

cc nikitaved pearu cpuhrsch amjames bhosmer

[ghstack-poisoned]
@pytorchmergebot
Copy link
Collaborator

Successfully rebased gh/alexsamardzic/12/orig onto refs/remotes/origin/viable/strict, please pull locally before adding more changes (for example, via ghstack checkout https://github.com/pytorch/pytorch/pull/101339)

pytorchmergebot pushed a commit that referenced this pull request May 17, 2023
…e linear operator

ghstack-source-id: bcb0b9672f4fc6b2c75a0fd2e39d6edca1a5db68
Pull Request resolved: #101339
@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

@izaitsevfb
Copy link
Contributor

@pytorchbot revert -m "Depends on #100881, which has to be reverted due to internal build breakage." -c ghfirst

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

@pytorchmergebot
Copy link
Collaborator

@alexsamardzic your PR has been successfully reverted.

pytorchmergebot added a commit that referenced this pull request May 18, 2023
…ed sparse linear operator (#101339)"

This reverts commit bfb3941.

Reverted #101339 on behalf of https://github.com/izaitsevfb due to Depends on #100881, which has to be reverted due to internal build breakage. ([comment](#101339 (comment)))
jcaip pushed a commit that referenced this pull request May 23, 2023
@alexsamardzic alexsamardzic reopened this Jun 4, 2023
…tured sparse linear operator"

cc nikitaved pearu cpuhrsch amjames bhosmer

[ghstack-poisoned]
alexsamardzic added a commit that referenced this pull request Jun 4, 2023
…e linear operator

ghstack-source-id: 1f4a2a1b7d7de4c4f25cbecb92ad6ac85309feeb
Pull Request resolved: #101339
@izaitsevfb
Copy link
Contributor

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

@cpuhrsch
Copy link
Contributor

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Rebase failed due to Command git -C /home/runner/work/pytorch/pytorch rebase refs/remotes/origin/viable/strict gh/alexsamardzic/12/orig returned non-zero exit code 1

Rebasing (1/2)
Auto-merging aten/src/ATen/native/native_functions.yaml
Auto-merging test/forward_backward_compatibility/check_forward_backward_compatibility.py
CONFLICT (content): Merge conflict in test/forward_backward_compatibility/check_forward_backward_compatibility.py
Auto-merging test/test_sparse.py
error: could not apply 66d2638bc35... Implement adding bias vector into structured sparse linear operator
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply 66d2638bc35... Implement adding bias vector into structured sparse linear operator

Raised by https://github.com/pytorch/pytorch/actions/runs/5284684057

…tured sparse linear operator"

cc nikitaved pearu cpuhrsch amjames bhosmer

Differential Revision: [D46453476](https://our.internmc.facebook.com/intern/diff/D46453476)

[ghstack-poisoned]
alexsamardzic added a commit that referenced this pull request Jun 16, 2023
…e linear operator

ghstack-source-id: cdb106bf3e9fa37467038573e54fb6a4f85ece8b
Pull Request resolved: #101339
@alexsamardzic
Copy link
Collaborator Author

@pytorchbot merge

@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/alexsamardzic/12/head branch June 20, 2023 14:16
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 module: sparse Related to torch.sparse open source release notes: sparse release notes category Reverted topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants