fangfangli / cleartk

Automatically exported from code.google.com/p/cleartk
0 stars 0 forks source link

WindowFeature and WindowNGramFeature should be deprecated #327

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
There's a couple places where these are used that are unrelated to their 
respective extractors - but these could be changed.  

Original issue reported on code.google.com by phi...@ogren.info on 13 Aug 2012 at 3:23

GoogleCodeExporter commented 9 years ago
This didn't get done before the 1.2 release. We'll take care of it in 1.3.

Original comment by steven.b...@gmail.com on 16 Aug 2012 at 1:05

GoogleCodeExporter commented 9 years ago

Original comment by lee.becker on 17 Feb 2013 at 6:05

GoogleCodeExporter commented 9 years ago

Original comment by phi...@ogren.info on 28 Mar 2013 at 4:33

GoogleCodeExporter commented 9 years ago
There are some tests related to the NameNumber debacle that use WindowFeature - 
so we shouldn't remove these two classes until that issue is resolved.

Original comment by phi...@ogren.info on 28 Mar 2013 at 4:35

GoogleCodeExporter commented 9 years ago
I have deprecated these classes in rev 48d9f6a45f9fb84eeee770788d38e5b17a223fda 
and updated the only dependency that was still using these classes: 
DefaultOutcomeFeatureExtractor.  I am now changing the name of this issue to 
reflect what needs to be done now - remove these classes after the next 
release.  

Original comment by phi...@ogren.info on 28 Mar 2013 at 6:13