elementdiscovery icon indicating copy to clipboard operation
elementdiscovery copied to clipboard

Issue with the "findOverlapping3" function

Open kulkarnip0 opened this issue 11 years ago • 3 comments

As discussed I just created the new issue for the problem. Below is the small snippet of code.

findOverlapping3('ds.nextround.prevdets',find(ds.classperbatch==ds.uniquelabels(dsidx)),[ds.batchfordetr(:,1),ds.classperbatch(ds.batchfordetr(:,2))],struct('ndetsforoverlap',.5,'maxoverlaps',3,'clusterer','agglomerative'))

I get below error in the log file:

sort by image compute overlaps ismapreducer:1 Undefined variable "internal" or class "internal.stats.getParamVal". file: '/data/chercheurs/kulkarnip/dev/elementdiscovery-master/linkage2.m' name: 'linkage2' line: 154

file: [1x73 char] name: 'findOverlapping3' line: 189

file: [1x82 char] name: 'dsmapredrun' line: 5

file: [1x83 char] name: 'dsmapreducer' line: 316

file: [1x90 char] name: 'dsmapreducerbarrier' line: 10

file: [1x87 char] name: 'dsmapreducerwrap' line: 6 MATLAB:undefinedVarOrClass

checkpassed =

1 Could you please help me resolve this problem.

Thanks Praveen

kulkarnip0 avatar Sep 15 '14 09:09 kulkarnip0

I am finding real hard time to alter 'findOverlapping3 to return a trivial clustering'. Is there possibility if some part can be commented or any other easy way.

kulkarnip0 avatar Sep 15 '14 12:09 kulkarnip0

I thought commenting the code from line number 104 to 205 in "findOverlapping3.m" and returning below code snippet should solve the problem.

detids=detids();%(ord(eliminate))=[]; detids(eliminate)=[]; scores=scores();%(ord(eliminate))=[]; scores(eliminate)=[];

kulkarnip0 avatar Sep 15 '14 12:09 kulkarnip0

I don't think your changes do what you want--the only output argument that gets read is component which is created on line 197.

I'd recommend reverting your changes and then adding the following two lines after line 130:

component=[c(detids(ord)) c(detids(:)*0+detids(ord(1))) c(scores(ord))];
return;

cdoersch avatar Sep 15 '14 20:09 cdoersch