Skip to content

Conversation

fmassa
Copy link
Member

@fmassa fmassa commented Oct 25, 2021

Depends on #4735

Summary: Track all detection model instantiation.

Reviewed By: sallysyw

Differential Revision: D31741603

fbshipit-source-id: 89467db5d89ea41c5a9ab95d1846264d0a1f4199

cc @datumbox

Summary: Track all detection model instantiation.

Reviewed By: sallysyw

Differential Revision: D31741603

fbshipit-source-id: 89467db5d89ea41c5a9ab95d1846264d0a1f4199
Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stamping

@@ -9,6 +9,8 @@
import torch
from torch import nn, Tensor

from ...utils import _log_api_usage_once
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Related failure: ImportError: cannot import name '_log_api_usage_once'

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is because this feature was added in #4735. I'll rebase this PR on top of master once I merge #4735

@fmassa fmassa merged commit f28d93b into pytorch:main Oct 25, 2021
@fmassa fmassa deleted the import_79cb1becc2d80a6e5b20810d97a1c2182d2a426f branch October 25, 2021 14:11
@github-actions
Copy link

Hey @fmassa!

You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

cyyever pushed a commit to cyyever/vision that referenced this pull request Nov 16, 2021
* Expand usage logging to detection models

Summary: Track all detection model instantiation.

Reviewed By: sallysyw

Differential Revision: D31741603

fbshipit-source-id: 89467db5d89ea41c5a9ab95d1846264d0a1f4199

* Fix lint

Co-authored-by: Kai Zhang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants