Nltk: Adopt abstract base class syntax

Created on 12 Sep 2015  ·  4Comments  ·  Source: nltk/nltk

NLTK has plenty of abstract base classes. Now that Python has support for these, we need to audit the codebase and adopt this syntax.

cf https://docs.python.org/3.0/whatsnew/2.6.html#pep-3119

goodfirstbug python3
Source
picture stevenbird

All 4 comments

Thanks @mrecachinas and @kmike for https://github.com/nltk/nltk/pull/1223. It looks like we have a model that can be applied across the toolkit now.

stevenbird picture stevenbird on 4 Jan 2016

@stevenbird Is the issue fixed?

souravsingh picture souravsingh on 19 May 2017
👍1

@stevenbird it seems that this issue has been fixed, it might be useful to close it.

gmotzespina picture gmotzespina on 12 Nov 2018
👍1

Thanks @souravsingh, @gmotzespina!

stevenbird picture stevenbird on 12 Nov 2018
Was this page helpful?
0 / 5 - 0 ratings

Related issues

StarWang picture StarWang  ·  5Comments
alvations picture alvations  ·  4Comments
talbaumel picture talbaumel  ·  4Comments
jeryini picture jeryini  ·  5Comments
Chris00 picture Chris00  ·  3Comments