Menu

#296 convenience methods for PropertyType.NumericalFingerprint

any future version
open
nobody
5
2018-02-05
2018-02-01
dim8
No

We should implement convenience methods to set and get integer arrays or ArrayLists for the PopertyType NumericalFingerprint in StringProperty (such a for BinFingerprint). Furthermore, we should implement a lenght method.

Old Request

The type allows to be processed only as String, although in the description is said to be of the form int, int, .... While this may be only in terms of values' meaning, extending it to actual ints would be of use. Because by being parsed as String, any such value cannot really be used in a meaningful way, e.g. to cluster or plot it. Proof in action: Try to plot the EStateNumericalFingerprint descriptor's value.

Discussion

  • Till Schäfer

    Till Schäfer - 2018-02-05

    The reason we do not store a actually a vector of integers is, that we need to store that data in a database column. Many databases do not even have such a type and you would need to map the vector to multiple rows, which makes such types a bit complex to handle properly. Furthermore, as long it is not absolutily necessary, we do not want to change the database layout in order to keep the database backward compatible.

    However, I would appreaciate a conveinience methods in the class StringProperty, such as we have done for the BitFingerprint type. Thus, we can create the methods getNumericalFingerprint, setNumericalFingerprint and getNumericalFingerprintLength.

    If you are actually working on a plugin or some other feature that uses the NumericalFingerprint type, please create such conveinice methods and create a seperate merge request. I will also keep the bug open for ourself. We could port the EStateNumericalFingerprinter to that methods.

     
  • Till Schäfer

    Till Schäfer - 2018-02-05
    • summary: PropertyType.NumericalFingerprint parsed as String --> conceinice methods for PropertyType.NumericalFingerprint
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1 +1,3 @@
    +We should implement conveinience methods to set and get integer arrays or ArrayLists for the PopertyType NumericalFingerprint in StringProperty (such a for BinFingerprint). Furthermore, we should implement a lenght method. 
    +# Old Request
     The type allows to be processed only as String, although in the description is said to be of the form `int, int, ...`.  While this may be only in terms of values' meaning, extending it to actual ints would be of use. Because by being parsed as String, any such value cannot really be used in a meaningful way,  e.g. to cluster or plot it. Proof in action: Try to plot the `EStateNumericalFingerprint` descriptor's value. 
    
     
  • Till Schäfer

    Till Schäfer - 2018-02-05
    • summary: conceinice methods for PropertyType.NumericalFingerprint --> convenience methods for PropertyType.NumericalFingerprint
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,3 +1,3 @@
    -We should implement conveinience methods to set and get integer arrays or ArrayLists for the PopertyType NumericalFingerprint in StringProperty (such a for BinFingerprint). Furthermore, we should implement a lenght method. 
    +We should implement convenience methods to set and get integer arrays or ArrayLists for the PopertyType NumericalFingerprint in StringProperty (such a for BinFingerprint). Furthermore, we should implement a lenght method. 
     # Old Request
     The type allows to be processed only as String, although in the description is said to be of the form `int, int, ...`.  While this may be only in terms of values' meaning, extending it to actual ints would be of use. Because by being parsed as String, any such value cannot really be used in a meaningful way,  e.g. to cluster or plot it. Proof in action: Try to plot the `EStateNumericalFingerprint` descriptor's value. 
    
     

Log in to post a comment.

MongoDB Logo MongoDB