-
Notifications
You must be signed in to change notification settings - Fork 161
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
Use RandomSource in a few more places #1599
Conversation
6fce4d2
to
cd9bcbb
Compare
Codecov Report
@@ Coverage Diff @@
## master #1599 +/- ##
==========================================
+ Coverage 64.45% 64.47% +0.02%
==========================================
Files 1003 1002 -1
Lines 326890 326805 -85
Branches 13216 13239 +23
==========================================
+ Hits 210696 210708 +12
+ Misses 113331 113234 -97
Partials 2863 2863
|
cd9bcbb
to
9ebe26d
Compare
I rebased this, fixed the test failures (I hope, we'll see when Travis completed) and added the previously missing documentation update for the Hopefully this can be merged soon -- @ChrisJefferson if you have any thoughts on this PR, please ... :-) |
9ebe26d
to
a03aa20
Compare
Also add test cases
a03aa20
to
358ebaa
Compare
Turns out initialization of the old-style |
7e74517
to
ba23ccf
Compare
This PR makes some progress on issue #1098.