Fix HYSPLIT reader to adjust for an arbitrary number of trajectories.

ARM-DOE / ACT

Atmospheric data Community Toolkit - A python based toolkit for exploring and analyzing time series atmospheric datasets

https://ARM-DOE.github.io/ACT/

Other

133 stars 34 forks source link

Fix HYSPLIT reader to adjust for an arbitrary number of trajectories. #835

Closed rcjackson closed 1 month ago

rcjackson commented 1 month ago

The HYSPLIT reader was assuming that the data started at line 13, which was breaking the reader when using other test files generated by students at the ARM Summer School. This now fixes that issue, making the HYSPLIT reader able to read files with an arbitrary number of trajectories.

[x] PEP8 Standards or use of linter
[x] Xarray Dataset or DataArray variable naming follows 'ds' or 'da' naming

AdamTheisen commented 1 month ago

@zssherman @rcjackson what are your thoughts on adding skiprows as an argument in case we run into different flavors of this in the future?

rcjackson commented 1 month ago

I don't think it would hurt any to just add that in as a keyword.

rcjackson commented 1 month ago

Went ahead and made the change @AdamTheisen.