search

Yale-LILY / SummerTime

An open-source text summarization toolkit for non-experts. EMNLP'2021 Demo
https://arxiv.org/abs/2108.12738
Apache License 2.0
264 stars 30 forks source link

Spacy Version Conflict #37

Closed chatc closed 3 years ago

chatc commented 3 years ago

SummTime uses Spacy 3 while HMNet uses Spacy 2. However, there are several differences between these two versions which are similar to Python 2 and Python 3. Hence, we need to either adapt one of two versions or find a way to use both of them.

niansong1996 commented 3 years ago

@chatc How hard is it to adapt HMNet to Spacy 3?

@troyfeng116 I think you dealt with a similar issue before?

troyfeng116 commented 3 years ago

For the most part, SpaCy is backwards compatible. @chatc are you having issues with pip installation, or with actually running on spacy==3x?

chatc commented 3 years ago

It’s not hard to adapt HMNet to Spacy 3. I will try to modify it to the Spacy 3 version. Thanks! @niansong1996

There are several running issues. I am sorry that I am not quite familiar with this package, but I found the following errors: nlp = spacy.load('en', parser = False) : TypeError: load() got an unexpected keyword argument 'parser' nlp.tagger.labels AttributeError: 'English' object has no attribute 'tagger' But it is okay since I am planning to change the version of Spacy in HMNet to 3 @troyfeng116