Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set a fixed random state in Louvain clustering #3553

Closed
wants to merge 1 commit into from

Conversation

mstrazar
Copy link
Contributor

Issue

Set a fixed random seed by default in Louvain clustering to promote reproducible results in scOrange.

Includes
  • Code changes
  • Tests
  • Documentation

@mstrazar mstrazar requested a review from lanzagar January 25, 2019 09:01
@codecov
Copy link

codecov bot commented Jan 25, 2019

Codecov Report

Merging #3553 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #3553      +/-   ##
=========================================
+ Coverage   83.79%   83.8%   +<.01%     
=========================================
  Files         370     370              
  Lines       66823   66823              
=========================================
+ Hits        55997   56002       +5     
+ Misses      10826   10821       -5

@lanzagar
Copy link
Contributor

Wasn't this fixed in #3492 ?
Are you trying to make it deterministic by default from scripts as well or is using the widget still not deterministic?

@mstrazar
Copy link
Contributor Author

Correct, I had another source of uncertainty somewhere.

@mstrazar mstrazar closed this Jan 25, 2019
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.

2 participants