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

Conversation

@abhinavarora
Copy link
Contributor

No description provided.

Copy link
Contributor

@parmeet parmeet left a comment

Choose a reason for hiding this comment

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

LGTM!

@codecov
Copy link

codecov bot commented Feb 22, 2022

Codecov Report

Merging #1626 (ccc3f79) into release/0.12 (73941c6) will increase coverage by 0.06%.
The diff coverage is 100.00%.

Impacted file tree graph

@@               Coverage Diff                @@
##           release/0.12    #1626      +/-   ##
================================================
+ Coverage         85.39%   85.45%   +0.06%     
================================================
  Files                47       47              
  Lines              2170     2179       +9     
================================================
+ Hits               1853     1862       +9     
  Misses              317      317              
Impacted Files Coverage Δ
torchtext/transforms.py 96.19% <100.00%> (+0.19%) ⬆️

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 73941c6...ccc3f79. Read the comment docs.

@parmeet parmeet merged commit 170b74a into pytorch:release/0.12 Feb 23, 2022
@parmeet parmeet changed the title Modify CLIPTokenizer to either infer number of merges from encoder json or take it in constructor (#1622) [Cherry-pick] Modify CLIPTokenizer to either infer number of merges from encoder json or take it in constructor (#1622) Feb 23, 2022
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.

3 participants