Skip to content
This repository was archived by the owner on Sep 10, 2025. It is now read-only.

Conversation

@parmeet
Copy link
Contributor

@parmeet parmeet commented Jun 1, 2021

This PR introduces usage of iopath in download_from_url and cleans up the function by removing redundant logic flow.

@parmeet parmeet changed the title [WIP] Streamlining download functionality using iopath Streamlining download functionality using iopath Jun 7, 2021
@codecov
Copy link

codecov bot commented Jun 28, 2021

Codecov Report

Merging #1319 (76035fa) into master (0435df1) will increase coverage by 1.69%.
The diff coverage is 53.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1319      +/-   ##
==========================================
+ Coverage   76.85%   78.55%   +1.69%     
==========================================
  Files          63       64       +1     
  Lines        3275     3315      +40     
==========================================
+ Hits         2517     2604      +87     
+ Misses        758      711      -47     
Impacted Files Coverage Δ
torchtext/_download_hooks.py 40.22% <40.22%> (ø)
torchtext/utils.py 91.66% <87.87%> (+18.61%) ⬆️
torchtext/vocab.py 92.74% <0.00%> (+25.80%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0435df1...76035fa. Read the comment docs.

Copy link
Contributor

@hudeven hudeven left a comment

Choose a reason for hiding this comment

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

LGTM

@parmeet parmeet merged commit 6cde431 into pytorch:master Jun 28, 2021
@parmeet parmeet deleted the download branch June 28, 2021 17:46
run:
- python
- requests
- iopath
Copy link
Member

Choose a reason for hiding this comment

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

Is this actually available on anaconda?

I can't seem to find it anywhere?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants