ClearTK / cleartk

Machine learning components for Apache UIMA
http://cleartk.github.io/cleartk/
Other
129 stars 58 forks source link

added featurename from named sub-extractor #434

Open admackin opened 7 years ago

admackin commented 7 years ago

previously feature name from sub-extractors in ClearTkExtractor were silently discarded, for reasons I can't discern (presumably accidental?). Compare with the out-of-bounds features a few lines below, which preserve the name from the sub-extractor. Please advise if this was in fact a principled decision (and I'll abandon the PR). If not, I can provide other supporting changes you need for this very simple patch