ahmetdursun / lire

Automatically exported from code.google.com/p/lire
Other
0 stars 0 forks source link

CompactCEDDQuant returns a 144 bit descriptor instead of a 60 bit descriptor #2

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

CEDD cedd = new CEDD();
cedd.Compact = true;
cedd.extract( ImageIO.read(new File("c://myfile.jpg")));
System.out.println(cedd.getStringRepresentation()); 

What is the expected output? What do you see instead?

desired:
cedd 60 5 3 1 3 1 0 0 0 0 0 1 2 0 0 2 0 0 0 0 0 4 6 0 3 4 0 0 0 3 0 1 3 0 1 4 0 
0 0 1 0 0 1 0 0 1 0 0 0 0 0 1 2 0 0 1 0 0 0 0 0

instead: (filling with 0s)
cedd 144 5 3 1 3 1 0 0 0 0 0 1 2 0 0 2 0 0 0 0 0 4 6 0 3 4 0 0 0 3 0 1 3 0 1 4 
0 0 0 1 0 0 1 0 0 1 0 0 0 0 0 1 2 0 0 1 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 
0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 
0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0

What version of the product are you using? On what operating system?
0.9.4 (trunk)

Please provide any additional information below.

If thats how it should be the tanimoto similarity is slower than having only to 
loop 60 bits

Original issue reported on code.google.com by g.ortega...@gmail.com on 29 May 2013 at 11:38

GoogleCodeExporter commented 9 years ago
version is 0.9.3

Original comment by g.ortega...@gmail.com on 29 May 2013 at 11:53

GoogleCodeExporter commented 9 years ago
Thanks for pointing that out. I was not aware of the compact version of CEDD as 
I always used the "non compact" one. I'll look into that.

cheers,
Mathias

Original comment by mathias....@gmail.com on 19 Jun 2013 at 8:06

GoogleCodeExporter commented 9 years ago
Identified the problem. I'll come up with a solution in the next few days. Most 
likely I'll re-wrap the compact version of CEDD in another class and make the 
compact member protected, not to be set from "outside".

Original comment by mathias....@gmail.com on 19 Jun 2013 at 8:30