search

nltk / nltk_book

NLTK Book
http://www.nltk.org/book
405 stars 143 forks source link

Fix issue about Earley parsers #27

Closed ewan-klein closed 10 years ago

ewan-klein commented 10 years ago

Trux makes a couple of correct observations about references to Earley parsers:

240. "Feature based grammars are parsed in NLTK using an Earley 
chart parser (see Section 9.5 for more information about this)."

"Section 9.5 does not provide information on Earley chart parsers."

254. (exercise 7)
"Develop a wrapper for the earley_parser so that a trace is only printed if the input sequence fails to parse."

"Note that I could not find a single instance of 'earley_parser' in the whole NLTK source code."

stevenbird commented 10 years ago

We do have parse.EarleyChartParser, so how about we just tweak those two references to Earley parsers.

stevenbird commented 10 years ago

Resolved in https://github.com/nltk/nltk_book/commit/502de57da886c5b981ab6ea2da4409ecd8208a3a