Skip to content
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

Assert spans <= max_seq_len #1302

Merged
merged 2 commits into from
Apr 8, 2021

Conversation

zphang
Copy link
Collaborator

@zphang zphang commented Mar 27, 2021

No description provided.

@zphang zphang requested a review from jeswan as a code owner March 27, 2021 06:52
@zphang zphang force-pushed the edge_probing_seq_len_assert branch 2 times, most recently from 24780e7 to d39fc12 Compare March 27, 2021 07:04
@zphang zphang force-pushed the edge_probing_seq_len_assert branch from d39fc12 to 738df91 Compare March 27, 2021 07:07
@codecov
Copy link

codecov bot commented Mar 27, 2021

Codecov Report

Merging #1302 (997128a) into master (4ab0c08) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1302   +/-   ##
=======================================
  Coverage   48.46%   48.47%           
=======================================
  Files         163      163           
  Lines       11214    11216    +2     
=======================================
+ Hits         5435     5437    +2     
  Misses       5779     5779           
Impacted Files Coverage Δ
jiant/tasks/lib/templates/edge_probing_two_span.py 98.71% <100.00%> (+0.03%) ⬆️

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 4ab0c08...997128a. Read the comment docs.

@jeswan jeswan merged commit b4b5de0 into nyu-mll:master Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants