Skip to content

Conversation

@scottstanie
Copy link
Member

Closes #682

Testing:

$ grep max_num_compressed  dolphin_config_many_compressed.yaml
  max_num_compressed: 3
  
  [INFO|sequential|L88] 2026-02-05T12:33:20-0500: Number of ministacks of size 4: 15
[INFO|sequential|L114] 2026-02-05T12:33:20-0500: Processing 4 SLCs. Output folder: dolphin_config_many_compressed/phase_linking/linked_phase/20200101_20200206
[INFO|single|L89] 2026-02-05T12:33:20-0500: VRTStack(4 bands, outfile=/Users/staniewi/Documents/Learning/papers/paper-opera-tgrs-disp-methods/notebooks/synthetic-example-exponential_decorrelation_tau60/dolphin_config_many_compressed/phase_linking/linked_phase/20200101_20200206.vrt): from [datetime.datetime(2020, 1, 1, 0, 0)] to [datetime.datetime(2020, 2, 6, 0, 0)]
[INFO|single|L108] 2026-02-05T12:33:20-0500: Processing 4 SLCs + 0 compressed SLCs.
[INFO|single|L113] 2026-02-05T12:33:20-0500: Total stack size (in pixels): (4, 1480, 2131)
[INFO|single|L108] 2026-02-05T12:33:25-0500: Processing 4 SLCs + 1 compressed SLCs.
[INFO|single|L108] 2026-02-05T12:33:30-0500: Processing 4 SLCs + 2 compressed SLCs.
[INFO|single|L108] 2026-02-05T12:33:35-0500: Processing 4 SLCs + 3 compressed SLCs.
[INFO|single|L108] 2026-02-05T12:33:40-0500: Processing 4 SLCs + 3 compressed SLCs.
...

(without this, the ministack also grows indefinitely as reported)

Checklist

  • The pull request title is a good summary of the changes - it will be used in the changelog
  • Unit tests for the changes exist
  • Tests pass on CI
  • Documentation reflects the changes where applicable
  • My PR is ready to review

@mirzaees mirzaees self-requested a review February 11, 2026 16:25
Copy link
Collaborator

@mirzaees mirzaees left a comment

Choose a reason for hiding this comment

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

Nice! thank you @scottstanie

@mirzaees mirzaees merged commit 3490079 into isce-framework:main Feb 11, 2026
5 checks passed
@scottstanie scottstanie deleted the fix-max-compressed-bug branch February 11, 2026 16:52
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.

Memory problem for PL step on compute cluster: max_num_compressed not respected

2 participants