BLLIP / bllip-parser

BLLIP reranking parser (also known as Charniak-Johnson parser, Charniak parser, Brown reranking parser) See http://pypi.python.org/pypi/bllipparser/ for Python module.
http://bllip.cs.brown.edu/
227 stars 53 forks source link

NLTK API tweakage. #23

Closed jimwhite closed 9 years ago

jimwhite commented 9 years ago

When I tried to use the Python ParsingShell I had errors that seem to be due to an API change in NLTK. nltk.tree.Tree.parse has apparently been removed/replaced by fromstring. There is bracket_parse too but its code says don't use it. Don't know if this is backward compatible but it works for me.

jimwhite commented 9 years ago

If you have Docker installed you can see the Python wrapper and GUI shell in action with a minimum of typing. The Docker Hub repo is here and can be launched with:

docker run -it --rm -p 5901:5901 jimwhite/bllip-parser-python

There are a few notes on running it in the description and there is a README.txt within the container (with basically the same info). I plan to make a posting and maybe a little video but figured I'd make a note of this development here in the meantime.

dmcc commented 9 years ago

Thanks for the patch! For compatibility, can we have it try fromstring and fallback to trying parse? I'm still using older versions of NLTK in some places.

jimwhite commented 9 years ago

Sure thing, should have done that to start with but I hardly ever code in Python. Since I'm not set up to test the old code I do that test first as before and the new stuff is in the exception block. The Docker build is updated now too (I tweaked the terminal settings so they're readable).