Skip to content

Conversation

fmassa
Copy link
Member

@fmassa fmassa commented Oct 25, 2021

Summary: We would like to track the internal usage of TorchVision components. We're tracking datasets usage now. This diff expand the tracking to all models.

Reviewed By: fmassa

Differential Revision: D31441632

fbshipit-source-id: e26072e582ac9f832c2056307ebf0eccf2ed6c9c

cc @datumbox

Summary: We would like to track the internal usage of TorchVision components. We're tracking [datasets usage](https://fburl.com/daiquery/dqpmemn3) now. This diff expand the tracking to all models.

Reviewed By: fmassa

Differential Revision: D31441632

fbshipit-source-id: e26072e582ac9f832c2056307ebf0eccf2ed6c9c
Copy link
Contributor

@prabhat00155 prabhat00155 left a comment

Choose a reason for hiding this comment

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

Stamping.

Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

LGTM.

I assume that quantized, detection etc will be added in follow up PRs.

@fmassa
Copy link
Member Author

fmassa commented Oct 25, 2021

@datumbox yes, in #4736 and #4737

@fmassa fmassa merged commit 5c57f5e into pytorch:main Oct 25, 2021
@fmassa fmassa deleted the import_6ab6aad98b283a625e98327249f515f2a273a7b1 branch October 25, 2021 13: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
Summary: We would like to track the internal usage of TorchVision components. We're tracking [datasets usage](https://fburl.com/daiquery/dqpmemn3) now. This diff expand the tracking to all models.

Reviewed By: fmassa

Differential Revision: D31441632

fbshipit-source-id: e26072e582ac9f832c2056307ebf0eccf2ed6c9c

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.

5 participants