Currently the code under EchoData.update_platform is very long, include a couple private functions and some sections that could be factored out. I suggest the following:
Create a echopype/echodata/utils_platform.py to store private utility functions used by update_platform. This follows the same pattern as combine.py that contains code used in combine_echodata
Move _extvar_properties and _clip_by_time_dim to utils_platform.py to keep the main code block concise.
Currently the code under
EchoData.update_platformis very long, include a couple private functions and some sections that could be factored out. I suggest the following:echopype/echodata/utils_platform.pyto store private utility functions used byupdate_platform. This follows the same pattern ascombine.pythat contains code used incombine_echodata_extvar_propertiesand_clip_by_time_dimtoutils_platform.pyto keep the main code block concise.utils_platform.pyfor the following code blocks that creates and cleans upmappings_expanded(i.e., the output of the new function will bemappings_expandedthat can be used) https://github.com/praneethratna/echopype/blob/ee7bcf0f0ce6cd744d74190e7bc46ba885e35792/echopype/echodata/echodata.py#L405-L463