Fail to add SpacyProcessor

zhanyuanucb commented 3 years ago

Describe the bug Encounter zipfile.BadZipFile: File is not a zip file while initializing a Pipeline with a SpacyProcessor.

To Reproduce Steps to reproduce the behavior:

from forte.pipeline import Pipeline
from forte.spacy.spacy_processors import SpacyProcessor
from forte.data.readers import TerminalReader

spacy_config_1 = {
    "processors": "sentence, umls_link",
    "lang": "en_ner_bionlp13cg_md",
    "use_gpu": False
}
test_pipeline = Pipeline().set_reader(TerminalReader()).add(SpacyProcessor(), config=spacy_config_1).initialize()

Environment (please complete the following information):

OS: Ubuntu
Version 16.04.7 LTS
Python and Package verions: Python 3.6.13, Forte 0.1.1, Spacy 2.3.5, Scispacy 0.3.0

hunterhector commented 3 years ago

Did you try this in a fresh environment? I try the example and it works on my side.

Here, the File is not a zip file since to be that the en-ner-bionlp13cg-md-0.3.0 model is not downloaded. Could you paste the full stack trace to help debug?

hunterhector commented 3 years ago

btw, SpacyProcessor are now at fortex.spacy. We have to change the name due to a problem we find.

https://github.com/asyml/forte-wrappers/commit/a9e5127c84757426e56781c7ad4d6c05f77e781a

asyml / forte-wrappers

Fail to add SpacyProcessor #66