mitrevf / airhead-research

Automatically exported from code.google.com/p/airhead-research
0 stars 0 forks source link

RETAIN_DOCUMENT_SPACE_PROPERTY has wrong suffix attached #75

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. run LatentSemanticAnalysis.java with a parameter
props.put(LatentSemanticAnalysis.RETAIN_DOCUMENT_SPACE_PROPERTY, "true");
2.
3.

What is the expected output? What do you see instead?
Document space should be retained after processSpace().
Instead, the following exception is thrown:

java.lang.IllegalArgumentException: 
edu.ucla.sspace.lsa.LatentSemanticAnalysis.dimensions is not an integer: true
    at edu.ucla.sspace.lsa.LatentSemanticAnalysis.processSpace(LatentSemanticAnalysis.java:503)

What version of the product are you using? On what operating system?
OS: Windows 7
Version: building from code, using revision 1107

Please provide any additional information below.

At LatentSemanticAnalysis.java, line 214 RETAIN_DOCUMENT_SPACE_PROPERTY is 
assigned:

public static final String RETAIN_DOCUMENT_SPACE_PROPERTY =
        PROPERTY_PREFIX + ".dimensions";

where ".dimensions" is the property suffix for LSA_DIMENSIONS_PROPERTY.

Original issue reported on code.google.com by an.velin...@gmail.com on 24 Oct 2010 at 11:29

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r1108.

Original comment by David.Ju...@gmail.com on 27 Oct 2010 at 6:37