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
151 stars 37 forks source link

Enhancement: Adding ability to pass list of files and/or xarray datasets into ARM Data Validator #851

Open cgodine opened 3 months ago

cgodine commented 3 months ago

Description

Currently, ARM Data Validator (ADV) takes an argument of either a filename string or xarray Dataset and returns a list of error messages. If a user needs to parse multiple files, it might be easier to pass a list of filenames and/or Datasets and have them returned in a dict where the keys are either the filename or Dataset and the value is the list of error messages.

zssherman commented 3 months ago

I think that's a good idea!

AdamTheisen commented 2 months ago

Hi All, just as a background the initial use case for this was so that a user could check the format of their files. Normally, I would assume that they would use the same script to write out their data to files so running through a list would be redundant for that use case. This function was really not meant to run through ARM's data, which will likely fail a lot of these more recent requirements.

This might take a fair amount of work to update. I would say for this function it's a low priority for the development team to work on.