Skip to content

Conversation

goggle
Copy link
Contributor

@goggle goggle commented Jun 1, 2019

I have noticed, that some of the code examples in the Quick start section of the README don't work because they were designed for an older version of Julia. This PR modifies these examples to make them work with Julia v1.1.1.

I have noticed, that some of the code examples in the `Quick start`
section of the README don't work because they were designed for an
older version of Julia. This PR modifies these examples to make them
work with Julia v1.1.1.
@coveralls
Copy link

coveralls commented Jun 1, 2019

Coverage Status

Coverage remained the same at 81.717% when pulling 44e3ae5 on goggle:readme-eye into 3b1c73f on JuliaArrays:master.

Copy link
Member

@andyferris andyferris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this @goggle - long overdue!

README.md Outdated
# Is (partially) hooked into BLAS, LAPACK, etc:
rand(MMatrix{20,20}) * rand(MMatrix{20,20}) # large matrices can use BLAS
eig(m3) # eig(), etc uses specialized algorithms up to 3×3, or else LAPACK
eigvecs(m3) # eigvecs(), etc uses specialized algorithms up to 3×3, or else LAPACK
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI the equivalent thing is eigen. (It probably doesn't matter).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I didn't find this, I only found eigvecs and eigvals. I will change it to eigen.

@goggle
Copy link
Contributor Author

goggle commented Jun 3, 2019

Let me know If I need to change anything else.
I can also squash the commits if you would like.

@c42f c42f merged commit b81044a into JuliaArrays:master Jun 3, 2019
@c42f c42f mentioned this pull request Jun 6, 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.

4 participants