search

laito / cleartk

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

POS tagger work #364

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
The POS tagger in cleartk-token should be revisited so that it conforms to the 
same type-system agnostic pattern that Lee's ClearNLP wrapper uses with a 
TokenOps interface.  

Also, we should have an eval implementation and train up some models besides.

Original issue reported on code.google.com by phi...@ogren.info on 16 Apr 2013 at 8:25

GoogleCodeExporter commented 9 years ago 
The *Ops classes should be moved to type system.

We should add @Beta annotations to all the *Ops classes.

Original comment by steven.b...@gmail.com on 24 Oct 2013 at 8:12

GoogleCodeExporter commented 9 years ago 
Ok - I don't think I will be able to rework the POS tagger before the end of 
the month.  I would like to propose that we deprecate the current code to 
communicate that it is going to be completely reworked and then move this issue 
to 2.1.

Original comment by phi...@ogren.info on 8 Dec 2013 at 3:31

GoogleCodeExporter commented 9 years ago 
Sounds good. It seems a bit weird to have something @Deprecated in 2.0, since 
we purposefully removed all @Deprecated APIs, so I would either rip the current 
code out, or mark it as @Beta.

Original comment by steven.b...@gmail.com on 8 Dec 2013 at 3:48

GoogleCodeExporter commented 9 years ago 
Well, marking it as @Beta seems even weirder considering that its deprecated!  
I think ripping it out is problematic because there is no migration path to the 
new as-of-yet-written part-of-speech tagger.  

The api wasn't deprecated in 1.4.1 but is in 2.0 - I don't see any problem with 
having newly deprecated code in 2.0.

Original comment by phi...@ogren.info on 8 Dec 2013 at 4:45

GoogleCodeExporter commented 9 years ago

Original comment by phi...@ogren.info on 15 Mar 2014 at 5:43