[SciPy-Dev] matlab reading / writing changes - review anyone?
Matthew Brett
matthew.brett at gmail.com
Tue Oct 12 23:40:06 EDT 2010
Hi,
>> I'm sorry - I probably have probably made the comparison a little too
>> large to be taken in in an instant, but I'd be very grateful for
>> feedback before merge.
>
> Based on a very shallow glance, looks mostly good to me, although:
>
> - Using UserWarning sounds a bit nasty -- maybe use
> class MatlabReaderWarning(UserWarning): pass instead?
Thanks, good idea.
> - Is there a test for "varmats_from_mat"?
No, only the doctest, thanks for the reminder, I'll put some in.
> - Breaks on Python 3:
Aha - that's sort of what I was expecting ;) At last I will set
myself up with a python 3 rig and test it properly.
Thanks a lot for the feedback, it's always cheering,
See you,
Matthew
More information about the SciPy-Dev
mailing list