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.

Sign up for GitHub

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

Jump to bottom

"Matrix samples" section of README is outdated #15

Open
garrison opened this issue Nov 13, 2014 · 2 comments
Open

"Matrix samples" section of README is outdated #15

garrison opened this issue Nov 13, 2014 · 2 comments

Comments

@garrison
Copy link

The README references things like GaussianHermiteMatrix, GaussianLaguerreMatrix, etc. However, in commit 929cab8 these methods were replaced and the API was changed. The README should be updated to reflect these changes.
jiahao added a commit that referenced this issue Apr 28, 2016
@jiahao
Modernize Wigner GaussianHermite code
c75e471 
Make β a type parameter, facilitating dispatch

Put documentation in docstrings where possible

Tests!

Ref: #15
@garrison
Copy link
Author

I was about to file a new issue regarding missing things in the README, but thought it made more sense to follow up here. In particular, neither Haar nor randfast is mentioned in the README. "I remember being annoyed at how hard this was to find," said Ralph_Smith in the Discourse thread on Random Orthogonal Matrices.

@dlfivefifty
Copy link
Member

If you make a PR I can merge it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@garrison @dlfivefifty