search

barzerman / barzer

barzer engine code
MIT License
2 stars 0 forks source link

SPELLCORRECT: REGRESSION #568

Closed nchepanov closed 11 years ago

nchepanov commented 11 years ago

I was testing ev_all user from eu.barzer.net query: купить в перми

with previous version of barzer we had: before="перми" after="пермь" and the city entity was matched correctly

with latest version of barzer: before="перми" after="пером" and wrong entity from different subclass was matched

the same behavior is now online here http://eu.barzer.net/sandbox/ev_all/

barzerman commented 11 years ago

this was caused by switching to 'featured' correction. before stemCorrection was done first and if it yielded result , it was accepted. right now we only get lev-based feature correction which sort of scuks as we dont depress suffixes I am going to add stem correction to the current spell correction first

barzerman commented 11 years ago

no, actually FeaturedMatchComparator::operator() computes levdist from a mixed case string . thats why Пермь loses out .. will fix that

barzerman commented 11 years ago

i got the fix. taking ownership of the ticket

barzerman commented 11 years ago

first fix was a fuckup

barzerman commented 11 years ago

deployed to venik