madlib-dev mailing list archives

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

    https://github.com/apache/madlib/pull/204#discussion_r152419042
  
    --- Diff: src/ports/postgres/modules/knn/knn.py_in ---
    @@ -88,12 +88,28 @@ def knn_validate_src(schema_madlib, point_source, point_column_name,
point_id,
             plpy.error("kNN Error: Data type '{0}' is not a valid type for"
                        " column '{1}' in table '{2}'.".
                        format(col_type_test, test_id, test_source))
    +
    +    fn_dist = fn_dist.lower().strip()
    --- End diff --
    
    I have added a check here. If fn_dist is None, set it to default 'squared_dist_norm2'
function. 


---

Mime
View raw message