Thread: Re: [Classifier4j-devel] WordProbability Refactor...
Status: Beta
Brought to you by:
nicklothian
From: Peter L. <pe...@le...> - 2003-07-14 01:30:49
|
Heya, I found it very frustrating trying to produce a patch using SF's CVS service. Constantly getting error 0 or EOF meaning there are currently too many connections to the server. Pete On Mon, 14 Jul 2003 10:46:24 +0930, "Nick Lothian" wrote: > > Thanks Pete, > > I've applied that patch - now sourceforge CVS seems to > be down, so I'll have > to update that later. > > ----- Original Message ----- > From: "Peter Leschev" <pe...@le...> > To: "Nick Lothian" <ni...@ma...>; > <cla...@li...> > Sent: Sunday, July 13, 2003 12:46 PM > Subject: Re: [Classifier4j-devel] WordProbability > Refactor... > > > > Heya, > > > > here are the files that I submitted to > sf.net for [ 768157 ] > > WordProbability refactor - Now in the unified > format... > > > > Pete > > > > ----- Original Message ----- > > From: "Nick Lothian" <ni...@ma...> > > To: <cla...@li...> > > Sent: Wednesday, July 09, 2003 11:29 PM > > Subject: Re: [Classifier4j-devel] WordProbability > Refactor... > > > > > > > Pete, that looks good, and I think the commons-lang > dependancy is fine. > > > > > > I'm having a little trouble applying it, though. I > don't do a lot of > > > CVS/Patch work, so it might be something I'm doing > wrong. > > > > > > I'm using Eclipse, which has a nice apply-patch > wizard, but unfortunatly > > > that will only process patches in unified format. > Could I get you to > redo > > > the patch using that format? > > > > > > Just send it to the list (and CC it to nick at > mackmo dot com in case it > > > won't accept attachments). > > > > > > Nick > > > > > > ----- Original Message ----- > > > From: "Peter Leschev" <pe...@le...> > > > To: <cla...@li...> > > > Sent: Wednesday, July 09, 2003 11:04 AM > > > Subject: [Classifier4j-devel] WordProbability > Refactor... > > > > > > > > > > > > I would recommend changing the > IWordsDataSource to > > > > > > return WordProbability objects instead of > double. > > > > This > > > > > > would ensure that BayesianClassifier doesn't > have to > > > > > > know how to create WordProbability Objects, > it just > > > > > > gets them from the IWordsDataSource. > > > > > > > > > > > > It would be nice if WordProbability knew how > to > > > > > > calculate it's own probability, given the > number of > > > > > > nonmatching & matching counts. This will > reduce > > > > > > duplication of code with new IWordsDataSource > > > > > > implementations. > > > > > > > > > > > > > > > Sounds pretty reasonable. I'm not very happy > with the > > > > use of the > > > > > WordProbability object at the moment, anyway. > > > > > > > > Heya, > > > > > > > > I've submitted a patch with this change. > > > > > > > > > > > > > > http://sourceforge.net/tracker/index.php?func=detail&aid=768157&group_id=79523&atid=556879 > > > > > > > > Pete > > > > > > > > > > > > > ------------------------------------------------------- > > > > This SF.Net email sponsored by: Parasoft > > > > Error proof Web apps, automate testing & more. > > > > Download & eval WebKing and get a free book. > > > > www.parasoft.com/bulletproofapps > > > > _______________________________________________ > > > > Classifier4j-devel mailing list > > > > Cla...@li... > > > > > https://lists.sourceforge.net/lists/listinfo/classifier4j-devel > > > > > > > > > > > > > > > > > ------------------------------------------------------- > > > This SF.Net email sponsored by: Parasoft > > > Error proof Web apps, automate testing & more. > > > Download & eval WebKing and get a free book. > > > www.parasoft.com/bulletproofapps > > > _______________________________________________ > > > Classifier4j-devel mailing list > > > Cla...@li... > > > > https://lists.sourceforge.net/lists/listinfo/classifier4j-devel > > > > > > > > > > > > ------------------------------------------------------- > This SF.Net email sponsored by: Parasoft > Error proof Web apps, automate testing & more. > Download & eval WebKing and get a free book. > www.parasoft.com/bulletproofapps1 > _______________________________________________ > Classifier4j-devel mailing list > Cla...@li... > https://lists.sourceforge.net/lists/listinfo/classifier4j-devel |
From: Nick L. <ni...@ma...> - 2003-07-14 11:43:12
|
Glad I'm not alone, then. Those changes are checked in now. ----- Original Message ----- From: "Peter Leschev" <pe...@le...> To: <ni...@ma...> Cc: <cla...@li...> Sent: Monday, July 14, 2003 11:00 AM Subject: Re: [Classifier4j-devel] WordProbability Refactor... > Heya, > > I found it very frustrating trying to produce a > patch using SF's CVS service. Constantly getting error > 0 or EOF meaning there are currently too many > connections to the server. > > Pete > > On Mon, 14 Jul 2003 10:46:24 +0930, "Nick Lothian" > wrote: > > > > Thanks Pete, > > > > I've applied that patch - now sourceforge CVS seems to > > be down, so I'll have > > to update that later. > > |
From: Peter L. <pe...@le...> - 2003-07-15 09:22:25
|
Heya, could you please verify that you've added the directory src/test/net/sf/classifier4J/bayesian to cvs? You need to perform a 'cvs add src/test/net/sf/classifier4J/bayesian', and then add all the files in that directory (and then commit). Thanks, Pete ----- Original Message ----- From: "Nick Lothian" <ni...@ma...> To: <cla...@li...> Sent: Monday, July 14, 2003 9:43 PM Subject: Re: [Classifier4j-devel] WordProbability Refactor... > Glad I'm not alone, then. > > Those changes are checked in now. > > ----- Original Message ----- > From: "Peter Leschev" <pe...@le...> > To: <ni...@ma...> > Cc: <cla...@li...> > Sent: Monday, July 14, 2003 11:00 AM > Subject: Re: [Classifier4j-devel] WordProbability Refactor... > > > > Heya, > > > > I found it very frustrating trying to produce a > > patch using SF's CVS service. Constantly getting error > > 0 or EOF meaning there are currently too many > > connections to the server. > > > > Pete > > > > On Mon, 14 Jul 2003 10:46:24 +0930, "Nick Lothian" > > wrote: > > > > > > Thanks Pete, > > > > > > I've applied that patch - now sourceforge CVS seems to > > > be down, so I'll have > > > to update that later. > > > > > > > ------------------------------------------------------- > This SF.Net email sponsored by: Parasoft > Error proof Web apps, automate testing & more. > Download & eval WebKing and get a free book. > www.parasoft.com/bulletproofapps1 > _______________________________________________ > Classifier4j-devel mailing list > Cla...@li... > https://lists.sourceforge.net/lists/listinfo/classifier4j-devel > > |
From: Nick L. <ni...@ma...> - 2003-07-15 10:44:34
|
Yes it is there (See http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/classifier4j/Classifier4J/src/test/net/sf/classifier4J/bayesian/ ) . If you are using the anonymous access to CVS I think sourceforge says it may be up to 24 hours old (although I have seen it older at times) because they run the anon access off different CVS servers or something. Anyway, I've added you as a developer to the classifier4J project (pleschev, right?), so you should have read-only CVS access via the developer CVS servers: :extssh:ple...@cv...:/cvsroot/classifier4J Let me know if that doesn't work for you. Nick ----- Original Message ----- From: "Peter Leschev" <pe...@le...> To: "Nick Lothian" <ni...@ma...>; <cla...@li...> Sent: Tuesday, July 15, 2003 6:51 PM Subject: Re: [Classifier4j-devel] WordProbability Refactor... > Heya, > > could you please verify that you've added the directory > src/test/net/sf/classifier4J/bayesian to cvs? You need to perform a 'cvs add > src/test/net/sf/classifier4J/bayesian', and then add all the files in that > directory (and then commit). > > Thanks, > Pete > > ----- Original Message ----- > From: "Nick Lothian" <ni...@ma...> > To: <cla...@li...> > Sent: Monday, July 14, 2003 9:43 PM > Subject: Re: [Classifier4j-devel] WordProbability Refactor... > > > > Glad I'm not alone, then. > > > > Those changes are checked in now. > > > > ----- Original Message ----- > > From: "Peter Leschev" <pe...@le...> > > To: <ni...@ma...> > > Cc: <cla...@li...> > > Sent: Monday, July 14, 2003 11:00 AM > > Subject: Re: [Classifier4j-devel] WordProbability Refactor... > > > > > > > Heya, > > > > > > I found it very frustrating trying to produce a > > > patch using SF's CVS service. Constantly getting error > > > 0 or EOF meaning there are currently too many > > > connections to the server. > > > > > > Pete > > > > > > On Mon, 14 Jul 2003 10:46:24 +0930, "Nick Lothian" > > > wrote: > > > > > > > > Thanks Pete, > > > > > > > > I've applied that patch - now sourceforge CVS seems to > > > > be down, so I'll have > > > > to update that later. > > > > > > > > > > > > ------------------------------------------------------- > > This SF.Net email sponsored by: Parasoft > > Error proof Web apps, automate testing & more. > > Download & eval WebKing and get a free book. > > www.parasoft.com/bulletproofapps1 > > _______________________________________________ > > Classifier4j-devel mailing list > > Cla...@li... > > https://lists.sourceforge.net/lists/listinfo/classifier4j-devel > > > > > > > |
From: Peter L. <pe...@le...> - 2003-07-15 13:06:04
|
Heya, > If you are using the anonymous access to CVS I think sourceforge says it may > be up to 24 hours old (although I have seen it older at times) because they > run the anon access off different CVS servers or something. Ahh... > Anyway, I've added you as a developer to the classifier4J project (pleschev, > right?), so you should have read-only CVS access via the developer CVS > servers: > > :extssh:ple...@cv...:/cvsroot/classifier4J > > Let me know if that doesn't work for you. Brilliant! Worked like a charm, Thanks! Pete |