Skip to content

Conversation

pmeier
Copy link
Collaborator

@pmeier pmeier commented Apr 22, 2021

Fixes #3708. If possible lets accept this for now without tests. I need to find a way to emulate this situation in the test suite.

@pmeier pmeier requested a review from datumbox April 22, 2021 11:15
@datumbox datumbox removed their request for review April 22, 2021 15:27
@datumbox
Copy link
Contributor

@pmeier sorry, I don't have lots of context around this specific fix. Is there a specific reason you prefer me to review it?

@pmeier pmeier requested a review from fmassa April 23, 2021 11:29
@pmeier
Copy link
Collaborator Author

pmeier commented Apr 23, 2021

@datumbox Yeah sorry, didn't really pay attention that you only labeled the issue.

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.

Stamping, there doesn't see to be much risk merging this (provided we wrap the next call as proposed below)

@fmassa fmassa merged commit 4150cea into master Apr 23, 2021
@fmassa fmassa deleted the fix-quota-exceeded branch April 23, 2021 15:43
facebook-github-bot pushed a commit that referenced this pull request May 4, 2021
Summary:
* fix check for exceeded quota on Google Drive

* safeguard quota exceed check

Reviewed By: NicolasHug

Differential Revision: D28169136

fbshipit-source-id: dc305a87e5936e142d1b9c73520f6b4f2e7764cf
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.

Unable to load CelebA dataset: "File is not zip file" error.
5 participants