Skip to content

fix jaccard similarity NaN#1324

Merged
Linary merged 1 commit intomasterfrom
fix-jaccard-similar-nan
Jan 7, 2021
Merged

fix jaccard similarity NaN#1324
Linary merged 1 commit intomasterfrom
fix-jaccard-similar-nan

Conversation

@zhoney
Copy link
Copy Markdown
Contributor

@zhoney zhoney commented Jan 6, 2021

Change-Id: I78a3f3da2be5480712f4545fbc93daad77f0bf1d

Change-Id: I78a3f3da2be5480712f4545fbc93daad77f0bf1d
int unionNum = CollectionUtil.union(set1, set2).size();
if (unionNum == 0) {
return 0.0D;
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correct title typo

@zhoney zhoney changed the title fix jaccard similarity nan fix jaccard similarity NaN Jan 6, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 6, 2021

Codecov Report

Merging #1324 (049263a) into master (26ac664) will increase coverage by 0.43%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1324      +/-   ##
============================================
+ Coverage     61.88%   62.32%   +0.43%     
- Complexity     5751     5772      +21     
============================================
  Files           377      377              
  Lines         31543    31543              
  Branches       4419     4419              
============================================
+ Hits          19521    19658     +137     
+ Misses        10029     9887     -142     
- Partials       1993     1998       +5     
Impacted Files Coverage Δ Complexity Δ
.../baidu/hugegraph/backend/query/ConditionQuery.java 86.40% <0.00%> (+0.40%) 106.00% <0.00%> (+1.00%)
.../backend/store/cassandra/CassandraSessionPool.java 57.14% <0.00%> (+1.02%) 11.00% <0.00%> (ø%)
...h/backend/store/scylladb/ScyllaDBTablesWithMV.java 96.03% <0.00%> (+96.03%) 10.00% <0.00%> (+10.00%)
...graph/backend/store/scylladb/ScyllaDBFeatures.java 100.00% <0.00%> (+100.00%) 3.00% <0.00%> (+3.00%)
.../backend/store/scylladb/ScyllaDBStoreProvider.java 100.00% <0.00%> (+100.00%) 7.00% <0.00%> (+7.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 26ac664...049263a. Read the comment docs.

@Linary Linary merged commit 52a8aae into master Jan 7, 2021
@Linary Linary deleted the fix-jaccard-similar-nan branch January 7, 2021 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants