Skip to content

Conversation

@apkille
Copy link
Contributor

@apkille apkille commented Jun 1, 2025

Note that I replaced the method rand(d::GaussianLaguerre, n::dims) with rand(d::GaussianLaguerre, n::Int) to match similar rand methods for other random matrix ensembles in the package. Ping @dlfivefifty for review.

@apkille
Copy link
Contributor Author

apkille commented Jun 1, 2025

Fixes #87

@codecov
Copy link

codecov bot commented Jun 1, 2025

Codecov Report

Attention: Patch coverage is 95.45455% with 1 line in your changes missing coverage. Please review.

Project coverage is 42.65%. Comparing base (fc98f3b) to head (29255e0).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/GaussianEnsembles.jl 95.45% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
+ Coverage   39.80%   42.65%   +2.84%     
==========================================
  Files          12       12              
  Lines         819      830      +11     
==========================================
+ Hits          326      354      +28     
+ Misses        493      476      -17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dlfivefifty dlfivefifty merged commit 37f177b into JuliaMath:master Jun 2, 2025
8 checks passed
@apkille apkille deleted the wishart branch June 2, 2025 13:09
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