richliao / textClassifier

Text classifier for Hierarchical Attention Networks for Document Classification
Apache License 2.0
1.07k stars 379 forks source link

Incorporating this model into tensorflow project #45

Open aaronrsun opened 4 years ago

aaronrsun commented 4 years ago

I am currently working on a project to improve code analysis using deep learning model. Because my project is based on tensorflow, I found myself having to make significant code change to this textClassifier in order to incorporate it. It turns out that this model improved the accuracy rate substantially. I am ready to contribute code back. Is there a suggestion on the best way to approach it because my changes have diverged from the current code base?

richliao commented 4 years ago

Hi aaronrsun, very nice to hear this little repo helped you with your project. If you like, you can cite my Github and this project. And It will be great if you can post your project link here. I would love to see how the Hierarchical network can help code analysis. Thanks.

aaronrsun commented 4 years ago

Hi richliao, thanks for the quick reply. I cited your project and paper in my work. The code analysis part isn't open source yet. I need to check with the person who own's that part of the code before making it available for you and others to see. Thank you for your patience. For the time being, I'm looking for a home for the modified version of your project with tensorflow dependencies. Maybe I should share it with you privately first so to hear your feedback.

richliao commented 4 years ago

Sorry, I have to be clear that I am not the author of the paper. I was merely interested in the paper and implemented it since the original paper didn't open source any implementation.