Skip to content

[FIX] Switch Sieve to DomainModel, which also fixes VizRank crash on meta attributes#1642

Merged
astaric merged 1 commit intobiolab:masterfrom
janezd:fix-sieve-rank2
Nov 18, 2016
Merged

[FIX] Switch Sieve to DomainModel, which also fixes VizRank crash on meta attributes#1642
astaric merged 1 commit intobiolab:masterfrom
janezd:fix-sieve-rank2

Conversation

@janezd
Copy link
Contributor

@janezd janezd commented Oct 7, 2016

Fixes #1546.
Breaks compatibility with old settings. See #1672 for a similar problem with the scatter plot.
Blocked by #1724 and #1727.

@codecov-io
Copy link

codecov-io commented Oct 7, 2016

Current coverage is 88.91% (diff: 100%)

Merging #1642 into master will not change coverage

@@             master      #1642   diff @@
==========================================
  Files            82         82          
  Lines          8847       8847          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           7866       7866          
  Misses          981        981          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 7b5f2be...50cc5ec

@janezd janezd changed the title [FIX] Switch Sieve to DomainModel, which also fixes VizRank crash on meta attributes [WIP] [FIX] Switch Sieve to DomainModel, which also fixes VizRank crash on meta attributes Oct 21, 2016
@janezd janezd force-pushed the fix-sieve-rank2 branch 2 times, most recently from c9bb195 to e3c8558 Compare November 11, 2016 14:35
@janezd janezd changed the title [WIP] [FIX] Switch Sieve to DomainModel, which also fixes VizRank crash on meta attributes [FIX] Switch Sieve to DomainModel, which also fixes VizRank crash on meta attributes Nov 11, 2016
@astaric
Copy link
Member

astaric commented Nov 18, 2016

I have rebased the commits onto the latest master as there were some conflicts.

@astaric astaric merged commit b6abc3b into biolab:master Nov 18, 2016
@janezd janezd deleted the fix-sieve-rank2 branch April 5, 2019 17:31
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