Skip to content

Conversation

@Jmennius
Copy link

@Jmennius Jmennius commented May 8, 2023

Just like #2258 but for modulated deformable convolutions - let torchvision do the heavylifting.
Support for modulated deform conv in torchvision exists since v0.9.0, so it doesn't seem to influence dependency requirements at the moment.
Updated tests to let most of them run on CPU and extended other tests to cover modulated variant as well.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 8, 2023
Jmennius added 2 commits May 8, 2023 14:08
Support for modulated deform conv in torchvision exists since v0.9.0 -
use that just like we do for regular deform conv.

Signed-off-by: Ievgen Popovych <jmennius@gmail.com>
…PU and CPU

Reorganized forward tests to execute on both GPU and CPU
for both variants of deformable conv.
Compare modulated deformable conv between CPU and GPU as well.
Also, let other tests run on CPU instead of limiting them to GPU machines.

Signed-off-by: Ievgen Popovych <jmennius@gmail.com>
@Jmennius Jmennius force-pushed the moduleted-deform-conv-cpu-via-torchvision branch from 11805fa to 738cce0 Compare May 8, 2023 11:08
@Jmennius
Copy link
Author

Jmennius commented May 8, 2023

Builds fail due to protobuf incompatibility with pytorch (too recent protobuf is installed for proto files in pytorch).
It should be pinned to ~=3.20 (or <4). For now it seems that pytorch doesn't update to support 4.* python protobuf (pytorch/pytorch#78362 and protobuf versions are pinned in CI).
Opened #4945 and #4939 with a fix.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants