Skip to content

Remove THFile, THDiskFile, and THMemoryFile#37830

Closed
ShawnZhong wants to merge 1 commit intopytorch:masterfrom
ShawnZhong:thfile
Closed

Remove THFile, THDiskFile, and THMemoryFile#37830
ShawnZhong wants to merge 1 commit intopytorch:masterfrom
ShawnZhong:thfile

Conversation

@ShawnZhong
Copy link
Copy Markdown
Contributor

@ShawnZhong ShawnZhong commented May 5, 2020

Fix #36996

Remove seemingly unused files.

@mruberry mruberry self-requested a review May 5, 2020 22:28
@mruberry mruberry added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label May 5, 2020
@mruberry
Copy link
Copy Markdown
Collaborator

mruberry commented May 5, 2020

Thanks for the PR, @ShawnZhong, is there a reason this is still a draft?

@ShawnZhong ShawnZhong marked this pull request as ready for review May 6, 2020 00:31
@ShawnZhong ShawnZhong force-pushed the thfile branch 2 times, most recently from 1b0207e to e2bcdba Compare May 6, 2020 17:21
@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented May 6, 2020

💊 CI failures summary and remediations

As of commit 0f704ee (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker.

See how this bot performed.

This comment has been revised 4 times.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Copy Markdown
Collaborator

@mruberry mruberry left a comment

Choose a reason for hiding this comment

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

Cool! Thanks for the PR, @ShawnZhong!

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@mruberry merged this pull request in 21ce433.

@ShawnZhong ShawnZhong deleted the thfile branch May 13, 2020 06:01
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Fix pytorch#36996

Remove seemingly unused files.
Pull Request resolved: pytorch#37830

Differential Revision: D21431969

Pulled By: mruberry

fbshipit-source-id: 824fa86f03ce0049ceddb093468800154ae8048b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Seemingly unused functions of TH: THDiskFile.h

4 participants