Alibaba-NLP / ACE

[ACL-IJCNLP 2021] Automated Concatenation of Embeddings for Structured Prediction
Other
300 stars 44 forks source link

Update to latest version of conllu #6

Closed EmilStenstrom closed 3 years ago

EmilStenstrom commented 3 years ago

?

nguyenbh commented 3 years ago

Thank you for submitting the PR. I closed it because there is not enough information on the PR to address the necessity of updating to the latest conllu.

EmilStenstrom commented 3 years ago

Thank you for submitting the PR. I closed it because there is not enough information on the PR to address the necessity of updating to the latest conllu.

Thanks for clarifying! There's no particular reason to update, simply because I am the author of the conllu package, and would like to see people only use the latest version of it if possible. It makes handling support requests easier.

So feel free to accept it or not!

wangxinyu0922 commented 3 years ago

Hi, thank you for the pull request. We will update the latest version of conllu in the next version of our code if there is no problem in running ACE with the latest conllu.