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

28 undefined names #41

Closed
cclauss opened this issue Jun 9, 2018 · 1 comment
Closed

28 undefined names #41

cclauss opened this issue Jun 9, 2018 · 1 comment

Comments

@cclauss
Copy link
Contributor

cclauss commented Jun 9, 2018

flake8 testing of https://github.com/huggingface/neuralcoref on Python 2.7.15

$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics

./neuralcoref/train/conll_processing_scripts/conll2name.py:305:16: F821 undefined name 'is_config_section_registered'
        if not is_config_section_registered(section):
               ^
./neuralcoref/train/conll_processing_scripts/conll2name.py:306:13: F821 undefined name 'on'
            on.common.log.status("Ignoring unknown configuration section", section)
            ^
./neuralcoref/train/conll_processing_scripts/conll2name.py:309:20: F821 undefined name 'is_config_registered'
            if not is_config_registered(section, option):
                   ^
./neuralcoref/train/conll_processing_scripts/conll2name.py:314:23: F821 undefined name 'allowed_config_values'
            allowed = allowed_config_values(section, option)
                      ^
./neuralcoref/train/conll_processing_scripts/conll2name.py:315:24: F821 undefined name 'allow_multiple_config_values'
            multiple = allow_multiple_config_values(section, option)
                       ^
./neuralcoref/train/conll_processing_scripts/conll2name.py:323:23: F821 undefined name 'required_config_options'
        for option in required_config_options(section):
                      ^
./neuralcoref/train/conll_processing_scripts/conll2name.py:332:9: F821 undefined name 'print_config_docs'
        print_config_docs()
        ^
./neuralcoref/train/conll_processing_scripts/conll2name.py:334:9: F821 undefined name 'on'
        on.common.log.status("Configuration Problems:")
        ^
./neuralcoref/train/conll_processing_scripts/conll2name.py:336:13: F821 undefined name 'on'
            on.common.log.status("  " + problem)
            ^
./neuralcoref/train/conll_processing_scripts/skeleton2conll.py:95:7: F821 undefined name 'insert_ignoring_dups'
      insert_ignoring_dups(cls, cursor, a_type)
      ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:321:16: F821 undefined name 'is_config_section_registered'
        if not is_config_section_registered(section):
               ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:322:13: F821 undefined name 'on'
            on.common.log.status("Ignoring unknown configuration section", section)
            ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:325:20: F821 undefined name 'is_config_registered'
            if not is_config_registered(section, option):
                   ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:330:23: F821 undefined name 'allowed_config_values'
            allowed = allowed_config_values(section, option)
                      ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:331:24: F821 undefined name 'allow_multiple_config_values'
            multiple = allow_multiple_config_values(section, option)
                       ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:339:23: F821 undefined name 'required_config_options'
        for option in required_config_options(section):
                      ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:348:9: F821 undefined name 'print_config_docs'
        print_config_docs()
        ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:350:9: F821 undefined name 'on'
        on.common.log.status("Configuration Problems:")
        ^
./neuralcoref/train/conll_processing_scripts/conll2coreference.py:352:13: F821 undefined name 'on'
            on.common.log.status("  " + problem)
            ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:320:16: F821 undefined name 'is_config_section_registered'
        if not is_config_section_registered(section):
               ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:321:13: F821 undefined name 'on'
            on.common.log.status("Ignoring unknown configuration section", section)
            ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:324:20: F821 undefined name 'is_config_registered'
            if not is_config_registered(section, option):
                   ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:329:23: F821 undefined name 'allowed_config_values'
            allowed = allowed_config_values(section, option)
                      ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:330:24: F821 undefined name 'allow_multiple_config_values'
            multiple = allow_multiple_config_values(section, option)
                       ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:338:23: F821 undefined name 'required_config_options'
        for option in required_config_options(section):
                      ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:347:9: F821 undefined name 'print_config_docs'
        print_config_docs()
        ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:349:9: F821 undefined name 'on'
        on.common.log.status("Configuration Problems:")
        ^
./neuralcoref/train/conll_processing_scripts/conll2parse.py:351:13: F821 undefined name 'on'
            on.common.log.status("  " + problem)
            ^
28    F821 undefined name 'is_config_section_registered'
28
@thomwolf
Copy link
Member

You are right, these scripts are from the CoNLL 2012 task website and are a bit out-dated now.

be5099e switch to the new script of Allen NLP.

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

No branches or pull requests

2 participants