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 mps support for maxpool3d #102148

Closed
wants to merge 2 commits into from
Closed

Add mps support for maxpool3d #102148

wants to merge 2 commits into from

Conversation

Berzeg
Copy link

@Berzeg Berzeg commented May 24, 2023

Fixes #100674

Added mps support for forward / backward passes of maxpool3d.

As mentioned in the referenced issue, I'm utilizing the maxpool 4d op that is available from the metal library to achieve this.

cc: @mattiaspaul

@pytorch-bot pytorch-bot bot added ciflow/mps Run MPS tests (subset of trunk) release notes: mps Release notes category labels May 24, 2023
@soulitzer soulitzer added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label May 25, 2023
@qqaatw
Copy link
Collaborator

qqaatw commented May 26, 2023

Hi, can you please add a test for it in test/test_mps.py?

Since there is an OpInfo for max_pool3d already, you can simply remove the following line and the consistency tests between cpu and mps will run automatically:

'nn.functional.max_pool3d': None,

@qqaatw
Copy link
Collaborator

qqaatw commented May 26, 2023

Also think you need to implement max_pool3d_with_indices* ops and adjust the decomp test in test_decomp.py since in MPS we treat w & w/o indices as separate kernels instead of sharing the same kernel and using CompositeImplicitAutograd. Let me know if you need any help!

@github-actions
Copy link

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

@github-actions github-actions bot added the Stale label Jul 25, 2023
@kulinseth
Copy link
Collaborator

Hi, can you please add a test for it in test/test_mps.py?

Since there is an OpInfo for max_pool3d already, you can simply remove the following line and the consistency tests between cpu and mps will run automatically:

'nn.functional.max_pool3d': None,

@Berzeg , thanks for the PR. Can you address comments from @qqaatw . Please let us know if you need help.

@qqaatw qqaatw removed the Stale label Aug 4, 2023
@github-actions
Copy link

github-actions bot commented Oct 3, 2023

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

@github-actions github-actions bot added the Stale label Oct 3, 2023
@github-actions github-actions bot closed this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/mps Run MPS tests (subset of trunk) open source release notes: mps Release notes category Stale 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.

Add support for MaxPool3D on the MPS backend
5 participants