Closed jeswan closed 4 years ago
Comment by pep8speaks Sunday Apr 12, 2020 at 21:04 GMT
Hello @pruksmhc! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
jiant/tasks/lm.py
:Line 195:41: W291 trailing whitespace Line 325:101: E501 line too long (117 > 100 characters) Line 348:101: E501 line too long (117 > 100 characters)
You can repair most issues by installing black and running: black -l 100 ./*
. If you contribute often, have a look at the 'Contributing' section of the README for instructions on doing this automatically.
Issue by pruksmhc Sunday Apr 12, 2020 at 21:04 GMT Originally opened as https://github.com/nyu-mll/jiant/pull/1064
Previously, we had done sentence-level MLM (https://github.com/nyu-mll/jiant/pull/1030). In this PR, we modify each example to be longer sequences.
pruksmhc included the following code: https://github.com/nyu-mll/jiant/pull/1064/commits