Skip to content

Conversation

sumanthratna
Copy link
Contributor

also see #5101

CC @prabhat00155

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Dec 16, 2021

💊 CI failures summary and remediations

As of commit 9839d03 (more details on the Dr. CI page):


None of the CI failures appear to be your fault 💚



🚧 1 ongoing upstream failure:

These were probably caused by upstream breakages that are not fixed yet.


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

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.

Thanks @sumanthratna for fixing this.

@prabhat00155 prabhat00155 merged commit f5dca44 into pytorch:main Dec 16, 2021
@github-actions
Copy link

Hey @prabhat00155!

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

@oke-aditya
Copy link
Contributor

I think this isn't classified as bug. As this feature isn't released yet. Previously I remember for masks_to_boxes we had same issue.

cc @datumbox and @prabhat00155

@NicolasHug
Copy link
Member

For such cases I tend to use the "other" label so that we can then bundle such PR with the original one in the release notes

@prabhat00155 prabhat00155 added other if you have no clue or if you will manually handle the PR in the release notes and removed bug labels Dec 19, 2021
facebook-github-bot pushed a commit that referenced this pull request Dec 21, 2021
Reviewed By: prabhat00155

Differential Revision: D33253478

fbshipit-source-id: 94c3c2696a73f8ee53a7324468779c4e61311c96
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/default cla signed module: utils other if you have no clue or if you will manually handle the PR in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants