zcpara / mate-tools

Automatically exported from code.google.com/p/mate-tools
0 stars 0 forks source link

Private methods disallow building a MATE wrapper #5

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Building a wrapper for the MATE pipeline for processing several documents and 
without loading each model for every document, one needs to call the out() 
methods of the respective tools (after initialising each tool once with a 
model).

However, the following methods are not set to public and thus disallow direct 
use of the is2.parser.Parser:

- is2.parser.Parser.out()
- is2.parser.Pipe.nextInstance() 

Also, using the morph tagger at is2.mtag.Tagger is not possible because of the 
non-public access to these fields:

- is2.mtag.Tagger.pipe
- is2.mtag.Tagger.params 

I do not believe this is intended as the respective fields/methods are public 
in the other processor classes, e.g. the lemmatizer or POS tagger.

The problem can easily be solved by setting these fields/methods to public.

For more details and links to the respective source code, also check 
http://korap.ids-mannheim.de/2013/07/issues-with-mate-pipeline/ 

Original issue reported on code.google.com by joa.bin...@gmail.com on 16 Jul 2013 at 12:56

GoogleCodeExporter commented 8 years ago
The idea was to use the apply method. We can have those public as well. 
Bernd

Original comment by boh...@informatik.uni-stuttgart.de on 5 Sep 2013 at 3:01