Skip to content

Pad parameters with different lengths across channel in vendor group#1702

Merged
leewujung merged 1 commit into
echostack-org:mainfrom
leewujung:vendor_param_pad
Jul 15, 2026
Merged

Pad parameters with different lengths across channel in vendor group#1702
leewujung merged 1 commit into
echostack-org:mainfrom
leewujung:vendor_param_pad

Conversation

@leewujung

Copy link
Copy Markdown
Member

This PR adds a short code section to pad parameters with different lengths across channel in the vendor group.

This was only encountered so far in an EA640 setup in which the lower freq channels contain a smaller number of pulse length choices than the higher freq channels.

@leewujung
leewujung requested a review from ctuguinay July 14, 2026 17:21
@leewujung leewujung added bug Something isn't working data conversion labels Jul 14, 2026

@ctuguinay ctuguinay left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good!

@leewujung
leewujung merged commit 70775aa into echostack-org:main Jul 15, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Echopype 2026 Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working data conversion

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants