Re: [Aramorph-users] A contribution for AraMorph

[Pierrick Brihaye]

Hi,

Ahmed El-dawy a écrit :

>   This is a piece of work I have made in the AraMorph project.

Nothing is actually attached AFAICS :-(

> This will speed up the start up of the system. On my machine the speed
> up is over 300%. Typically it was starting in 14.281 seconds. After my
> enhancement it is starting in 4.670 seconds!!
> My work is as follows
> InMemoryDictionaryHandler.java
> 1- Move the Pattern p declaration to the class level. This makes it
> created once.
> 2- Change the translation of the gloss so that it is made in one pass
> using a HashMap.
> 3- Used log4j package for outputting messages instead of System.out
> AraMorph.java
> 4- Just added a line for initializing the log4j.

All these changes look very interesting but in order to be efficient, I 
would prefer to have some patches.

This suppose that you have a local CVS version of Aramorph (an an open 
2401 port). Feel free to ask more if you have difficulties to set up 
your CVS client.

> I've attached the files I have changed and the log4j library for convenience.
> Please tell me whether you are adding this work to the project or not.

Since what you have done was in the TODOs, they are very likely to be 
added :-) ... ASAP.

> More contributions are coming in the horizon.

Fine ! One change that would greatly improve performance would be to 
have the choice of the dictionaries handlers. For now, we just have an 
InMemoryDictionaryHandler, but it would be nice to have a 
LuceneDirectoryHandler, a HSQLDictionaryHandler, a 
JDBCDictionaryHandler... all implementing a new DictionaryHandler interface.

Hear from you soon,

p.b.