Skip to content

[WIP][Do Not Review] enabling tests that depends on downloading from google drive #1180

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from
Closed

Conversation

parmeet
Copy link
Contributor

@parmeet parmeet commented Feb 18, 2021

Enabling back all the tests that depend on downloading data from google drive (essentially reversing what was done in #1171)

Our current understanding is that based on:

  1. we are caching .data folder (where all the data downloaded goes during testing) which is refreshed every week.
  2. Based on merge of [BC-breaking] Split raw sequence tagging datasets into individual files #1176, all the google drive data now have defined path in .data folder (defined using _PATH variable)
  3. If data is available in .data folder, it will used during testing (and will not trigger the download: refer
    return path
    )

Ideally this should ensure that we do not download if the data is already cached by CCI. Any blind spot here?

@parmeet
Copy link
Contributor Author

parmeet commented Feb 23, 2021

Closing it up, No longer relevant at the moment.

@parmeet parmeet closed this Feb 23, 2021
@parmeet parmeet deleted the gdrivequota branch February 23, 2021 00:56
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.

2 participants