Skip to content

SEADAS Band Grouping Improvements#593

Open
knowles wants to merge 9 commits intomasterfrom
SEADAS-band-group-14.0.0-2026-02-23
Open

SEADAS Band Grouping Improvements#593
knowles wants to merge 9 commits intomasterfrom
SEADAS-band-group-14.0.0-2026-02-23

Conversation

@knowles
Copy link
Contributor

@knowles knowles commented Feb 26, 2026

Modifications made to improve Band Grouping. Note a related (likely dependent) pull request appears in both snap-engine and snap-desktop. So all description is given here in this pull request.

Some example matches:

  1. "^a_" would match bandnames starting with "a_" (for example "a_442") and create a display folder named "a"
  2. "_unc$" would match bandnames ending with "unc" and create a display folder named "unc"
  3. "^chlor_a$" would match only "chlor_a" and create a display folder named "chlor_a"
  4. "Rrs" would match any bandnames containing "Rrs" and creates a display folder named "Rrs"
    Note in this scenario there is a boolean field:
    snap-core/src/main/java/eu/esa/snap/core/datamodel/group/BandGroupingPath.java line 16:
    boolean useStartsWithInsteadOfContains = false;
    SeaDAS sets this to "true" which changes the behavior to:
    "Rrs" would match any bandnames starting with "Rrs"
  5. "Rrs*" wildcards are still supported however are not supported in combination with "^" and "$":
    For example "^Rrs*" would ignore "^" as a match criteria.
    The display folder removes "" if at start or end of name, but not in the middle of the name:
    For example:
    "Rrs" creates folder "Rrs"
    "Rrs
    " creates folder "Rrs"
    but "Rrs
    unc" create folder "Rrs*unc"
    6 "Coords#latitude,longitude" would match bandnames "latitude" and "longitude" and create a display folder named "Coords".

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@knowles
Copy link
Contributor Author

knowles commented Feb 27, 2026

I added a small change and pushed it today in this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants