madlib-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jingyimei <...@git.apache.org>
Subject [GitHub] madlib pull request #195: Feature: Add grouping support to HITS
Date Wed, 15 Nov 2017 08:55:47 GMT
Github user jingyimei commented on a diff in the pull request:

    https://github.com/apache/madlib/pull/195#discussion_r151062713
  
    --- Diff: src/ports/postgres/modules/graph/hits.sql_in ---
    @@ -164,10 +168,20 @@ INSERT INTO edge VALUES
     (3, 0, 1),
     (4, 0, 1),
     (5, 6, 1),
    -(6, 3, 1);
    +(6, 3, 1),
    +(0, 1, 2),
    +(0, 2, 2),
    +(0, 4, 2),
    +(1, 2, 2),
    +(1, 3, 2),
    +(2, 3, 2),
    +(3, 0, 2),
    +(4, 0, 2),
    +(5, 6, 2),
    +(6, 3, 2);
     </pre>
     
    --# Compute the HITS scores:
    +-# Running HITS with default values for optional parameters:
    --- End diff --
    
    In those examples without grouping, duplicate edges from different groups are calculated
twice. We discussed this before and just wondering if we can add some explanation here(may
not be necessary tho).


---

Mime
View raw message