search

tensorflow / io

Dataset, streaming, and file system extensions maintained by TensorFlow SIG-IO
Apache License 2.0
702 stars 283 forks source link

Support for .MAT files #270

Open kmh4321 opened 5 years ago

kmh4321 commented 5 years ago

Hello,

This is a feature request which I feel will be useful to the community. .MAT Files are the default file formats used by MATLAB and would help researchers move around MATLAB and TensorFlow more freely. I would be glad to contribute in adding it but before raising a PR/working on it, I wanted to create this placeholder issue to discuss it first! I am also not sure about any legal issues or any IP related issues as MATLAB may have proprietary content / licenses around using the data format so I would appreciate any input I can get on that too.

yongtang commented 5 years ago

@kmh4321 I think this is definitely an area of interests for many community members. For tensorflow-io, it is licensed as Apache License 2.0 (same as TensorFlow). We could take any dependency as long as it does not conflict. So while MATLAB itself may be proprietary, as long as the code part that is build against (linked) from tensorflow-io is compatible with Apache then it should be fine.

Looking forward to further discussions or your PR!

kmh4321 commented 5 years ago

@yongtang Okay, thank you. I'll have a look at the current additions and start on the PR. I'll keep this issue open till I raise the PR and will reach out if I need any help/pointers.

Ouwen commented 5 years ago

@kmh4321 any updates on parsing matfiles?

kmh4321 commented 5 years ago

@Ouwen Unfortunately I haven't gotten around to it quiet yet, do you want to collaborate on this?

Ouwen commented 5 years ago

@kmh4321, unfortunately, no bandwidth at the moment. Just ping this thread if you decide to commit some time to it. I'll likewise do the same.