Speed too slow when writing mat73

ami-iit / matio-cpp

A C++ wrapper of the matio library, with memory ownership handling, to read and write .mat files.

https://ami-iit.github.io/matio-cpp/

BSD 2-Clause "Simplified" License

59 stars 9 forks source link

Speed too slow when writing mat73 #85

Open zhengliuer opened 2 weeks ago

zhengliuer commented 2 weeks ago

Dear all,

I successfully wrote my data to .mat v7.3, but I noticed the speed was extremely slow. To compare, the same data cost me less than 1sec to write a v5 .mat, but about 47sec for a v73.

Is this as expected? Has anyone faced the same problem with me? Any ideas to solve this?

I will keep trying to find out why and the solution of it.

Best, Zheng

traversaro commented 2 weeks ago

Indeed the time seems quite a lot, but if there is an issue, it may be more in matio https://github.com/tbeu/matio I guess, as matio-cpp only provides a C++ interface for the C functionalities provided in matio.

Anyhow, before opening an issue there, I think it make sense:

Check if you are using the Release version of libmatio and matio-cpp, and not for some reason the debug version.
If possible, provide a Minimal Reproducible Example of the issue (see https://stackoverflow.com/help/minimal-reproducible-example)

zhengliuer commented 2 weeks ago

Thanks! I will discuss the issue there.

zhengliuer commented 2 weeks ago

https://github.com/tbeu/matio/issues/261#issue-2654362703

I opened an issue in matio.

traversaro commented 2 weeks ago

I think that the mre should use matio, not matio-cpp, otherwise the mantainer does not know if the problem is in matio or matio-cpp. Furthermore, I would wait for having a working mre before opening an issue.

S-Dafarra commented 1 week ago

Hi @zhengliuer, did you happen to find the source of your issue?

zhengliuer commented 3 days ago

Hi @zhengliuer, did you happen to find the source of your issue?

Sorry, no. I was occupied by some other things, so I closed the issue for now. Now I have some time for this. And luckily, according to the test result from https://github.com/tbeu/matio/issues/261#issuecomment-2495400046, the performance degradation is confirmed.