OSOceanAcoustics / echopype

Enabling interoperability and scalability in ocean sonar data analysis
https://echopype.readthedocs.io/
Apache License 2.0
96 stars 72 forks source link

Revise command line tool using uniform interface #66

Closed leewujung closed 4 years ago

leewujung commented 4 years ago

Need to update the command line tool now that the convert and model modules have uniform interface for AZFP and EK60 data (#53).

Specifically, need to change the input optional parameter to accommodate the additional .XML needed for AZFP unpacking.

The goal is to be able to do:

In the above:

prarobinson commented 4 years ago

Currently ConvertAZFP expects two arguments: a data file and an xml file. Is the last warning above, “The XML file is ignored as it is not needed for converting", meant to be issued when the user specifies ek60, or is the goal to make xml files optional for azfp data?

leewujung commented 4 years ago

Ouch missed the second half of the sentence. It is for if user issued an XML file for unpacking EK60 data.