Skip to content

Conversation

@peterjc123
Copy link
Collaborator

Fixes #26998.

yn4bit and others added 2 commits October 1, 2019 10:52
Summary:
Fixes pytorch#26998
Pull Request resolved: pytorch#27047

Differential Revision: D17666050

Pulled By: ezyang

fbshipit-source-id: f02ebd5320ae25f8949be20d0744fe3cd3e2fee9
(cherry picked from commit 1afe3fc)
(cherry picked from commit 5846008)
@peterjc123 peterjc123 requested a review from ezyang October 1, 2019 03:04
@pytorchbot pytorchbot added module: internals Related to internal abstractions in c10 and ATen module: operators labels Oct 1, 2019
@pytorchbot pytorchbot added the module: tests Issues related to tests (not the torch.testing module) label Oct 1, 2019
@peterjc123
Copy link
Collaborator Author

peterjc123 commented Oct 1, 2019

@ezyang The test passes and i will revert the unrelated changes.

@peterjc123
Copy link
Collaborator Author

@pytorchbot rebase this please

@ezyang
Copy link
Contributor

ezyang commented Oct 1, 2019

Thank you for being on top of this. Waiting on CI to agree this works.

Copy link
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.

@ezyang is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ezyang merged this pull request in ec07d14.

@peterjc123 peterjc123 deleted the load_offset_fix branch October 2, 2019 02:04
soumith pushed a commit that referenced this pull request Oct 4, 2019
* Fixed seek offset size to 64bit. (#27047)

Summary:
Fixes #26998
Pull Request resolved: #27047

Differential Revision: D17666050

Pulled By: ezyang

fbshipit-source-id: f02ebd5320ae25f8949be20d0744fe3cd3e2fee9
(cherry picked from commit 1afe3fc)

* Use _lseeki64 instead for MSVC

(cherry picked from commit f49f78d)
pdlive215 pushed a commit to pdlive215/pytorch that referenced this pull request Nov 27, 2019
Summary:
Fixes pytorch#26998.
Pull Request resolved: pytorch#27125

Differential Revision: D17687154

Pulled By: ezyang

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

Labels

Merged module: internals Related to internal abstractions in c10 and ATen module: tests Issues related to tests (not the torch.testing module) open source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

torch.load() - OSError on a file larger than 2GB

6 participants