[ 
https://issues.apache.org/jira/browse/LUCENE-3767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13224551#comment-13224551
 ] 

Michael McCandless commented on LUCENE-3767:
--------------------------------------------

bq. I haven't backported the test for PositionLengthAttribute since the 
structure seems very different in branch_3x. Perhaps this is something we could 
look to as we start using PositionLength in more of the other analyzers?

Hmm you mean TestSimpleAttributeImpl?  I think that's fine.  We'll
improve testing of pos length as we make more tokenizers/filters
graphs...

bq. Does the svn:mergeinfo look okay in the patch?

Hmm looks spooky... there was a trick for this... (and I know it's
hard to merge back since the path changed in trunk).  Maybe do svn
propdel svn:mergeinfo on all affected files, and then at the top level
you can just do an "svn merge --record-only" to update toplevel
mergeinfo?

bq. There will be some minor discrepancies between trunk and branch_3x with 
regards to Set<?> and CharArraySet types. Perhaps we should use CharArraySet on 
across the board here in the future?

This was from LUCENE-3765; I think it's OK to require
CharArraySet to Kuromoji...

Looks like the added files are not included in the patch (eg
RollingCharBuffer)... if you're using svn >= 1.7, you can run "svn
diff --show-copies-as-adds" maybe?  Or, use
dev-tools/scripts/diffSources.py... but I don't think it's
really necessary before committing...

Otherwise patch looks great!

                
> Explore streaming Viterbi search in Kuromoji
> --------------------------------------------
>
>                 Key: LUCENE-3767
>                 URL: https://issues.apache.org/jira/browse/LUCENE-3767
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: modules/analysis
>            Reporter: Michael McCandless
>            Assignee: Christian Moen
>             Fix For: 3.6, 4.0
>
>         Attachments: LUCENE-3767.patch, LUCENE-3767.patch, LUCENE-3767.patch, 
> LUCENE-3767.patch, LUCENE-3767.patch, LUCENE-3767_branch_3x.patch, 
> SolrXml-5498.xml, compound_diffs.txt
>
>
> I've been playing with the idea of changing the Kuromoji viterbi
> search to be 2 passes (intersect, backtrace) instead of 4 passes
> (break into sentences, intersect, score, backtrace)... this is very
> much a work in progress, so I'm just getting my current state up.
> It's got tons of nocommits, doesn't properly handle the user dict nor
> extended modes yet, etc.
> One thing I'm playing with is to add a double backtrace for the long
> compound tokens, ie, instead of penalizing these tokens so that
> shorter tokens are picked, leave the scores unchanged but on backtrace
> take that penalty and use it as a threshold for a 2nd best
> segmentation...

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to