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

Constructor bug #7597

Closed
Jutho opened this issue Jul 14, 2014 · 4 comments
Closed

Constructor bug #7597

Jutho opened this issue Jul 14, 2014 · 4 comments

Comments

@Jutho
Copy link
Contributor

Jutho commented Jul 14, 2014

I discovered the following failed on me today. It must be recent, since this caused a recent error in the tests of my TensorOperations package which did execute fine before.

type WrapStridedArray{T,N}
    data::StridedArray{T,N}
end
A=randn(3,3)
isa(A,StridedArray{Float64,2}) # => true
WrapStridedArray{Float64,2}(A)
# => ERROR: type: WrapStridedArray: in new, expected Union(SubArray{Float64,2,A<:DenseArray{T,N},I<:(Union(Int64,Range{Int64})...,)},DenseArray{Float64,2}), got Array{Float64,2}
@quinnj
Copy link
Member

quinnj commented Jul 14, 2014

I don't see the error in 1a41588

@Jutho
Copy link
Contributor Author

Jutho commented Jul 14, 2014

My apologies, I should first have built the latest version.

@Jutho Jutho closed this as completed Jul 14, 2014
@quinnj
Copy link
Member

quinnj commented Jul 14, 2014

Note there were some recent changes to the provided default inner constructors in #7071; not sure if that would have resulted in the change of behavior for you.

@Jutho
Copy link
Contributor Author

Jutho commented Jul 14, 2014

I would think that I was running a julia version which was at most a few days old, but since I also forgot to check the versioninfo before pulling the latest, this could be the cause. I will remember for next time to first pull and build the latest version.

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

No branches or pull requests

2 participants