JuliaMath / JuliaMath/RandomMatrices.jl

remove camelcase in `rand(W::Haar, n::Int, doCorrection::Int = 1)`

Open
#94 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
Julia
Stars
94
Forks
41
PR merge metrics
No merged PRs in 30d

Description

Could probably just do `correct::Int = 1` instead. This should be a kwarg rather than an optional argument too.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at the `rand(W::Haar, n::Int, doCorrection::Int = 1)` entry point and inspect its surrounding method definition and call sites. Update the argument naming and argument style as requested, then verify that existing uses still work and the public signature reflects the intended keyword argument.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
backend
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.