sogou / SogouMRCToolkit

This toolkit was designed for the fast and efficient development of modern machine comprehension models, including both published models and original prototypes.
Apache License 2.0
746 stars 162 forks source link

Missing import sys and other issues #11

Open cclauss opened 5 years ago

cclauss commented 5 years ago

flake8 testing of https://github.com/sogou/SMRCToolkit on Python 3.7.1

$ flake8 . --count --select=E9,F63,F72,F82 --show-source --statistics

./sogou_mrc/dataset/squadv2.py:88:55: F632 use ==/!= to compare str, bytes, and int literals
            "answer_start": answer_token_starts[0] if len(answer_token_starts) > 0 is not None else None,
                                                      ^
./sogou_mrc/dataset/squadv2.py:89:51: F632 use ==/!= to compare str, bytes, and int literals
            "answer_end": answer_token_ends[0] if len(answer_token_ends) > 0 is not None else None,
                                                  ^
./sogou_mrc/dataset/coqa.py:363:21: F821 undefined name 'sys'
                    sys.stderr.write("Turn id should match index {}: {}\n".format(i + 1, qa))
                    ^
./sogou_mrc/dataset/coqa.py:368:25: F821 undefined name 'sys'
                        sys.stderr.write("Question turn id does match answer: {} {}\n".format(qa, answer))
                        ^
./sogou_mrc/dataset/coqa.py:372:21: F821 undefined name 'sys'
                    sys.stderr.write("Gold file has duplicate stories: {}".format(source))
                    ^
./sogou_mrc/libraries/tokenization.py:39:27: F821 undefined name 'unicode'
    elif isinstance(text, unicode):
                          ^
./sogou_mrc/libraries/tokenization.py:62:27: F821 undefined name 'unicode'
    elif isinstance(text, unicode):
                          ^
./sogou_mrc/libraries/modeling.py:364:10: F821 undefined name 'output'
  return output
         ^
2     F632 use ==/!= to compare str, bytes, and int literals
6     F821 undefined name 'sys'
8