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

Migrate multilabel_margin_loss_forward from the TH to Aten (CUDA) #24603

Closed
VitalyFedyunin opened this issue Aug 16, 2019 · 0 comments
Closed
Assignees
Labels
better-engineering Relatively self-contained tasks for better engineering contributors module: loss Problem is related to loss function module: porting Issues related to porting TH/THNN legacy to ATen native triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Comments

@VitalyFedyunin
Copy link
Contributor

Porting TH operators is essential for code simplicity and performance reasons.

Porting guides and Q&A are available in umbrella issue: #24507

Feel free to add @VitalyFedyunin as a reviewer to get a prioritized review.

@VitalyFedyunin VitalyFedyunin added better-engineering Relatively self-contained tasks for better engineering contributors module: operators module: porting Issues related to porting TH/THNN legacy to ATen native triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module labels Aug 16, 2019
@mruberry mruberry added module: loss Problem is related to loss function and removed module: operators (deprecated) labels Oct 10, 2020
@thomasjpfan thomasjpfan self-assigned this Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
better-engineering Relatively self-contained tasks for better engineering contributors module: loss Problem is related to loss function module: porting Issues related to porting TH/THNN legacy to ATen native triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants