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

Upstream Resnext3d from classivision #5463

Closed
3 tasks
jdsgomes opened this issue Feb 23, 2022 · 1 comment
Closed
3 tasks

Upstream Resnext3d from classivision #5463

jdsgomes opened this issue Feb 23, 2022 · 1 comment

Comments

@jdsgomes
Copy link
Contributor

jdsgomes commented Feb 23, 2022

🚀 The feature

  • Upstream resnext3d from classyvision
  • Replace conv3d + norm + activation blocks with the new Conv3dNormAct block
  • Run tests and make sure the performance is in line with the original implementation

Motivation, pitch

This effort will contribute to the [RFC] Batteries Included - Phase 2 initiative

Alternatives

No response

Additional context

No response

cc @datumbox

@jdsgomes
Copy link
Contributor Author

Closing this ticket because the classyvision implementation is not canonical and because we are focusing our efforts now on adding newer architectures such has MViT.

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

No branches or pull requests

2 participants