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.

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

Couple missing branches for sqrtm and solve #227

Merged
merged 1 commit into from
Jun 13, 2017

Conversation

kshyatt
Copy link
Collaborator

@kshyatt kshyatt commented Jun 13, 2017

No description provided.

@andyferris andyferris merged commit 9ccda4e into JuliaArrays:master Jun 13, 2017
@andyferris
Copy link
Member

Thanks :)

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 88.184% when pulling 84d8f75 on kshyatt:ksh/test into 39eccd8 on JuliaArrays:master.

@andyferris
Copy link
Member

Interesting inference regression on nightly...

@kshyatt kshyatt deleted the ksh/test branch August 3, 2017 22:58
oschulz pushed a commit to oschulz/StaticArrays.jl that referenced this pull request Apr 4, 2023
This introduces the internal utility `maybe_convert_elt` used to
convert values to the element type of the StructArray before
assignment. This fixes errors that are caused by assigning a value
that does not have the same fields as the struct but which can be
converted to such a type (e.g., `T<:Real` -> `Complex{T}`). Rather
than calling `convert` directly, `maybe_convert_elt` can be
specialized for particular types, like LazyRow, which should not be
converted.

Fixes JuliaArrays#131
Fixes JuliaArrays#216
Closes JuliaArrays#184

Co-authored-by: Gustavo Goretkin <[email protected]>
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.

3 participants