pysat / pysatCDF

Python reader for NASA CDF file format
BSD 3-Clause "New" or "Revised" License
10 stars 7 forks source link

Update _cdf.py #28

Closed gregstarr closed 3 years ago

gregstarr commented 3 years ago

adding a try/except to _process_return_multi_z_attr when decoding characters so that when uninterpretable characters are encountered, they are just replaced with some filler character but the user can still access the data.

gregstarr commented 3 years ago

see #26 for details

gregstarr commented 3 years ago

Hello, just wanted to ping this issue

rstoneback commented 3 years ago

Hello, just wanted to ping this issue

Thanks for the reminder. Checking it out now.

rstoneback commented 3 years ago

Changes to the latest Mac OS made it hard for me to get this compiled. The system prefers to use clang now instead of gcc. Had to track down a different machine.

I can confirm that pysatCDF can load SWARM L1B files as well as files it used to be able to load, like from C/NOFS VEFI.

I did also notice that to_pysat() function won't work for SWARM since measurement times are internally hard coded to 'epoch', rather than a user supplied string. to_pysat was originally designed for NASA's CDAWeb metadata standards. I'll post an issue #29

gregstarr commented 3 years ago

Thanks for looking into this, I made the suggested changes.