-
Notifications
You must be signed in to change notification settings - Fork 71
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
matwrite fails for a basic dictionary on an isbits test #162
Comments
It looks like MAT.jl doesn't support writing char to mat files yet. julia> matwrite("char.mat", Dict("chr" => 'a'))
ERROR: This is the write function for CompositeKind, but the input doesn't fit
Stacktrace:
[1] error(s::String)
@ Base .\error.jl:33
[2] m_write(mfile::MAT.MAT_HDF5.MatlabHDF5File, parent::HDF5.File, name::String, s::Char)
@ MAT.MAT_HDF5 C:\Users\woclass\.julia\packages\MAT\f523T\src\MAT_HDF5.jl:530
[3] write(parent::MAT.MAT_HDF5.MatlabHDF5File, name::String, thing::Char)
@ MAT.MAT_HDF5 C:\Users\woclass\.julia\packages\MAT\f523T\src\MAT_HDF5.jl:546
[4] matwrite(filename::String, dict::Dict{String, Char}; compress::Bool)
@ MAT C:\Users\woclass\.julia\packages\MAT\f523T\src\MAT.jl:157
[5] matwrite(filename::String, dict::Dict{String, Char})
@ MAT C:\Users\woclass\.julia\packages\MAT\f523T\src\MAT.jl:148
[6] top-level scope
@ REPL[43]:1 |
Coming back to this again, the surprising part of it is that when reading a single character string, MAT automatically parses this as a
|
I know this is far from being good, but whouldn't it be relevant to avoid this error with
This now results in a correct roundtrip from/to .mat files This would allow to close #143 as well. @simonster, what do you make of this ? |
Bump ? |
MAT.jl v0.10.1 seems to fail with this call
is there something brroken, or I am using this incorrectly ?
The text was updated successfully, but these errors were encountered: