noaa-oar-arl / monetio

The Model and ObservatioN Evaluation Tool I/O package
https://monetio.readthedocs.io
MIT License
16 stars 19 forks source link

Fix pardump reading on some platforms #101

Closed zmoon closed 1 year ago

zmoon commented 1 year ago

Resolves #99

In our testing, the issue seems to be that on Windows, dtype '>l' (l as in long) resolves to '>i4', but it is supposed to be '>i8' for this file format. On Linux, don't have this issue.

zmoon commented 1 year ago

@amcz could you maybe confirm that results are still what you expect with this change?

amcz commented 1 year ago

@amcz could you maybe confirm that results are still what you expect with this change?

Yes. I have checked and confirm that on the linux platform the reader still works as expected. Thanks.