Skip to content

Conversation

kotchin
Copy link
Contributor

@kotchin kotchin commented Nov 8, 2021

Fixes #4871 by reverting back the change introduced in #3939.

cc @pmeier

…et and is_valid_file rather than just the filename
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Nov 8, 2021

💊 CI failures summary and remediations

As of commit 6f82595 (more details on the Dr. CI page):


  • 1/1 failures introduced in this PR

1 failure not recognized by patterns:

Job Step Action
CircleCI binary_libtorchvision_ops_android Build 🔁 rerun

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.

@datumbox datumbox requested a review from pmeier November 9, 2021 08:39
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.

thanks @kotchin

Copy link
Collaborator

@pmeier pmeier left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @kotchin!

@github-actions
Copy link

github-actions bot commented Nov 9, 2021

Hey @datumbox!

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

@datumbox datumbox added module: datasets revert(ed) For reverted PRs, and PRs that revert other PRs bug labels Nov 9, 2021
facebook-github-bot pushed a commit that referenced this pull request Nov 15, 2021
…ake_dataset and is_valid_file rather than just the filename (#4885)

Summary: Co-authored-by: Alexander Unnervik <alex.unnervik@idiap.ch>

Reviewed By: datumbox

Differential Revision: D32298975

fbshipit-source-id: d6b2fc4bf14e05acf45ae77eb675032b1ea671ae
cyyever pushed a commit to cyyever/vision that referenced this pull request Nov 16, 2021
…et and is_valid_file rather than just the filename (pytorch#4885)

Co-authored-by: Alexander Unnervik <alex.unnervik@idiap.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug ciflow/default cla signed module: datasets revert(ed) For reverted PRs, and PRs that revert other PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide complete filepath to is_valid_file in make_dataset rather than only the filename

5 participants