nikitakit / self-attentive-parser

High-accuracy NLP parser with models for 11 languages.
https://parser.kitaev.io/
MIT License
861 stars 153 forks source link

Update base_parser.py #63

Closed michaeljohns2 closed 3 years ago

michaeljohns2 commented 4 years ago

Updated for tensorflow v1 or v2 compatibility.

michaeljohns2 commented 4 years ago

Tested on both tf 1.15 and 2.2 and logic worked.

nikitakit commented 3 years ago

This issue is fixed in benepar 0.1.3.

Starting with benepar 0.2.0a0, I've also switched to not using tensorflow at all. My original hope of having this package remain usable without any ongoing maintenance has sadly proven to be inadequate. Unfortunately modern ML code seems to have a short shelf-life -- meanwhile software like the EVALB program checked into this repo remains usable despite no changes over the course of a decade or more. With v0.2.0, I'm switching to a new approach where the code has more dependencies, but it's also easier to maintain because I'm using these same dependencies for my research and I have to keep up with software changes anyway. (This is different from tensorflow, which I don't use myself because I find pytorch and jax to better serve my research needs)