madlib-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From iyerr3 <...@git.apache.org>
Subject [GitHub] madlib pull request #204: Added additional distance metrics for k-NN: Jira-1...
Date Wed, 22 Nov 2017 00:15:30 GMT
Github user iyerr3 commented on a diff in the pull request:

    https://github.com/apache/madlib/pull/204#discussion_r152419848
  
    --- Diff: src/ports/postgres/modules/knn/knn.py_in ---
    @@ -89,20 +89,20 @@ def knn_validate_src(schema_madlib, point_source, point_column_name,
point_id,
                        " column '{1}' in table '{2}'.".
                        format(col_type_test, test_id, test_source))
     
    -    fn_dist = fn_dist.lower().strip()
    -    dist_functions = {
    -        schema_madlib + '.dist_norm1',
    -        schema_madlib + '.dist_norm2',
    -        schema_madlib + '.squared_dist_norm2',
    -        schema_madlib + '.dist_angle',
    -        schema_madlib + '.dist_tanimoto'}
    -
    -    if plpy.execute("""select prorettype != 'DOUBLE PRECISION'::regtype
    -            OR proisagg = TRUE AS OUTPUT from pg_proc where
    -            oid='{fn_dist}(DOUBLE PRECISION[], DOUBLE PRECISION[])'::regprocedure;"""
    -                    .format(**locals()))[0]['output'] or fn_dist not in dist_functions:
    -        plpy.error(
    -            "KNN error: Distance function has wrong signature or is not a simple function.")
    +    if fn_dist and fn_dist is not None:
    --- End diff --
    
    only need the `if fn_dist:`. 
    `fn_dist = None` is automatically processed as `False` in the `if` check. 


---

Mime
View raw message