From: Chris D. <ce...@ui...> - 2008-09-06 08:36:29
Attachments:
solr_proximity.patch
solr_boosting.patch
|
I noticed, at the vufind.org/demo site, that proximity searches break SOLR, e.g., http://vufind.org/demo/Search/Home?lookfor=%22economics+Keynes%22%7E10&type=all&submit=Find I'm not sure if this is fixed in the latest svn or not. I also noticed that my boosting searches weren't working correctly for my installation of VuFind. I created two tiny patches (based on rev. 1001) that seems to fix these -- at least for me. However, I must warn you that I am backporting code from the latest svn into a rather older version of VuFind, therefore YMMV. I just wanted to share my experience in case folks are having an issue with either of these types of searches and might want to take a closer look. Cheers, Chris |
From: Chris D. <ce...@ui...> - 2008-09-07 00:16:20
|
I got the regexp wrong in the patch below. > Index: web/sys/Solr.php > =================================================================== > --- web/sys/Solr.php (revision 1001) > +++ web/sys/Solr.php (working copy) > @@ -753,7 +753,7 @@ > public function tokenizeInput($input) > { > // Tokenize on spaces and quotes > - preg_match_all('/"[^"]*"|[^ ]+/', $input, $words); > + preg_match_all('/"[^"]*"[~[0-9]+]*|[^ ]+/', $input, $words); The above should be: preg_match_all('/"[^"]*"[~[0-9]+]*|"[^"]*"|[^ ]+/', $input, $words); instead. Sorry about that. Chris |
From: Chris D. <ce...@ui...> - 2008-09-09 15:10:35
|
I notice that the error exists at VU (which I think is based on svn, no?): http://library.villanova.edu/Find/Search/Home?lookfor="barack+obama"~5&type=&search=catalog&submit=Find I realize this is probably not the highest priority, but I was curious if this is still an issue in the latest svn. Thanks, Chris On Sat, Sep 06, 2008 at 07:16:28PM -0500, Chris Delis wrote: > > I got the regexp wrong in the patch below. > > > Index: web/sys/Solr.php > > =================================================================== > > --- web/sys/Solr.php (revision 1001) > > +++ web/sys/Solr.php (working copy) > > @@ -753,7 +753,7 @@ > > public function tokenizeInput($input) > > { > > // Tokenize on spaces and quotes > > - preg_match_all('/"[^"]*"|[^ ]+/', $input, $words); > > + preg_match_all('/"[^"]*"[~[0-9]+]*|[^ ]+/', $input, $words); > > > The above should be: > > preg_match_all('/"[^"]*"[~[0-9]+]*|"[^"]*"|[^ ]+/', $input, $words); > > instead. > > Sorry about that. > > Chris > > ------------------------------------------------------------------------- > This SF.Net email is sponsored by the Moblin Your Move Developer's challenge > Build the coolest Linux based applications with Moblin SDK & win great prizes > Grand prize is a trip for two to an Open Source event anywhere in the world > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > _______________________________________________ > Vufind-tech mailing list > Vuf...@li... > https://lists.sourceforge.net/lists/listinfo/vufind-tech |
From: Andrew N. <and...@vi...> - 2008-09-09 16:44:16
|
No - this is an error and hasn't been fixed yet. Andrew > -----Original Message----- > From: vuf...@li... [mailto:vufind-tech- > bo...@li...] On Behalf Of Chris Delis > Sent: Tuesday, September 09, 2008 11:11 AM > To: vuf...@li... > Subject: Re: [VuFind-Tech] "Proximity Searches" and "Boosting a Term" > Not Working? (PATCH) > > > I notice that the error exists at VU (which I think is based on svn, > no?): > > http://library.villanova.edu/Find/Search/Home?lookfor="barack+obama"~5& > type=&search=catalog&submit=Find > > I realize this is probably not the highest priority, but I was curious > if this is still an issue in the latest svn. > > Thanks, > Chris > > On Sat, Sep 06, 2008 at 07:16:28PM -0500, Chris Delis wrote: > > > > I got the regexp wrong in the patch below. > > > > > Index: web/sys/Solr.php > > > =================================================================== > > > --- web/sys/Solr.php (revision 1001) > > > +++ web/sys/Solr.php (working copy) > > > @@ -753,7 +753,7 @@ > > > public function tokenizeInput($input) > > > { > > > // Tokenize on spaces and quotes > > > - preg_match_all('/"[^"]*"|[^ ]+/', $input, $words); > > > + preg_match_all('/"[^"]*"[~[0-9]+]*|[^ ]+/', $input, > $words); > > > > > > The above should be: > > > > preg_match_all('/"[^"]*"[~[0-9]+]*|"[^"]*"|[^ ]+/', $input, $words); > > > > instead. > > > > Sorry about that. > > > > Chris > > > > --------------------------------------------------------------------- > ---- > > This SF.Net email is sponsored by the Moblin Your Move Developer's > challenge > > Build the coolest Linux based applications with Moblin SDK & win > great prizes > > Grand prize is a trip for two to an Open Source event anywhere in the > world > > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > > _______________________________________________ > > Vufind-tech mailing list > > Vuf...@li... > > https://lists.sourceforge.net/lists/listinfo/vufind-tech > > ----------------------------------------------------------------------- > -- > This SF.Net email is sponsored by the Moblin Your Move Developer's > challenge > Build the coolest Linux based applications with Moblin SDK & win great > prizes > Grand prize is a trip for two to an Open Source event anywhere in the > world > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > _______________________________________________ > Vufind-tech mailing list > Vuf...@li... > https://lists.sourceforge.net/lists/listinfo/vufind-tech |
From: Chris D. <ce...@ui...> - 2008-09-09 17:40:52
|
On Tue, Sep 09, 2008 at 12:44:16PM -0400, Andrew Nagy wrote: > No - this is an error and hasn't been fixed yet. Thanks. I added this to sourceforge bugtracker. BTW, the boosting terms seem to work fine. Chris > > Andrew > > > -----Original Message----- > > From: vuf...@li... [mailto:vufind-tech- > > bo...@li...] On Behalf Of Chris Delis > > Sent: Tuesday, September 09, 2008 11:11 AM > > To: vuf...@li... > > Subject: Re: [VuFind-Tech] "Proximity Searches" and "Boosting a Term" > > Not Working? (PATCH) > > > > > > I notice that the error exists at VU (which I think is based on svn, > > no?): > > > > http://library.villanova.edu/Find/Search/Home?lookfor="barack+obama"~5& > > type=&search=catalog&submit=Find > > > > I realize this is probably not the highest priority, but I was curious > > if this is still an issue in the latest svn. > > > > Thanks, > > Chris > > > > On Sat, Sep 06, 2008 at 07:16:28PM -0500, Chris Delis wrote: > > > > > > I got the regexp wrong in the patch below. > > > > > > > Index: web/sys/Solr.php > > > > =================================================================== > > > > --- web/sys/Solr.php (revision 1001) > > > > +++ web/sys/Solr.php (working copy) > > > > @@ -753,7 +753,7 @@ > > > > public function tokenizeInput($input) > > > > { > > > > // Tokenize on spaces and quotes > > > > - preg_match_all('/"[^"]*"|[^ ]+/', $input, $words); > > > > + preg_match_all('/"[^"]*"[~[0-9]+]*|[^ ]+/', $input, > > $words); > > > > > > > > > The above should be: > > > > > > preg_match_all('/"[^"]*"[~[0-9]+]*|"[^"]*"|[^ ]+/', $input, $words); > > > > > > instead. > > > > > > Sorry about that. > > > > > > Chris > > > > > > --------------------------------------------------------------------- > > ---- > > > This SF.Net email is sponsored by the Moblin Your Move Developer's > > challenge > > > Build the coolest Linux based applications with Moblin SDK & win > > great prizes > > > Grand prize is a trip for two to an Open Source event anywhere in the > > world > > > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > > > _______________________________________________ > > > Vufind-tech mailing list > > > Vuf...@li... > > > https://lists.sourceforge.net/lists/listinfo/vufind-tech > > > > ----------------------------------------------------------------------- > > -- > > This SF.Net email is sponsored by the Moblin Your Move Developer's > > challenge > > Build the coolest Linux based applications with Moblin SDK & win great > > prizes > > Grand prize is a trip for two to an Open Source event anywhere in the > > world > > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > > _______________________________________________ > > Vufind-tech mailing list > > Vuf...@li... > > https://lists.sourceforge.net/lists/listinfo/vufind-tech |