search

cbaziotis / datastories-semeval2017-task4

Deep-learning model presented in "DataStories at SemEval-2017 Task 4: Deep LSTM with Attention for Message-level and Topic-based Sentiment Analysis".
MIT License
196 stars 63 forks source link

`semeval_data_loader.py` missing #1

Closed laeubli closed 7 years ago

laeubli commented 7 years ago

Thumbs up for your code! Trying to reproduce your SemEval 2017 results for Subtask A, I realised that semeval_data_loader.py is missing in the dataset module. It's explicitly excluded from the repository via .gitignore.

If I'm not mistaken, there's no way to train a model without the SemEvalDataLoader class from semeval_data_loader.py. Would you mind sharing it?

cbaziotis commented 7 years ago

Hi. Yes i am aware of this. There are a few other things missing too... This project was a part of a bigger codebase and i am trying to untangle the projects and remove dependencies. I am going to push a new version in the next few days.

cbaziotis commented 7 years ago

@laeubli i pushed some fixes. Give it a go.

laeubli commented 7 years ago

Thanks much. The ftfy package seems to be missing from requirements.txt, but apart from that, things seem to be working now.

cbaziotis commented 7 years ago

@laeubli Nice! ftfy was a (missing) dependency of ekphrasis. I added it to it'srequirements.txt. Thanks for letting me know.