Skip to content

Update dist s1 entrypoints#3029

Open
cmarshak wants to merge 6 commits intoASFHyP3:developfrom
ACCESS-Cloud-Based-InSAR:update-dist-s1-entrypoints
Open

Update dist s1 entrypoints#3029
cmarshak wants to merge 6 commits intoASFHyP3:developfrom
ACCESS-Cloud-Based-InSAR:update-dist-s1-entrypoints

Conversation

@cmarshak
Copy link
Contributor

@cmarshak cmarshak commented Feb 18, 2026

Few questions:

  1. I am not sure if my command will be problematic without a character in front of run for aws batch.
  2. dist_s1_data option in OPERA_DIST_S1_CONFIRMATION is comma separated granules (could be up to 100 granules).

@cmarshak cmarshak requested review from a team as code owners February 18, 2026 18:26
@cmarshak
Copy link
Contributor Author

Copy link
Contributor

@jhkennedy jhkennedy left a comment

Choose a reason for hiding this comment

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

Overall, the job spec looks fine.

I don't see it added to any deployments. @cmarshak did you want to add to hyp3-tibet-jpl-test, hyp3-tibet-jpl, and hyp3-nisar-jpl like OPERA_DIST_S1?

- name: ''
image: ghcr.io/opera-adt/dist-s1
command:
- run
Copy link
Contributor

Choose a reason for hiding this comment

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

This is fine and will roughly translate to something that looks like:

docker run --rm -it ghcr.io/opera-adt/dist-s1:test run ...

cmarshak and others added 2 commits February 19, 2026 09:16
Co-authored-by: Joseph H Kennedy <me@jhkennedy.org>
Co-authored-by: Joseph H Kennedy <me@jhkennedy.org>
@cmarshak
Copy link
Contributor Author

Want to confirm that the array input for the API (e.g. this passes space separated granules to the CLI i.e.

API {'arg': [<granule_0>, <granule_1>]} --> CLI --arg <granule_0> <granule_1>

That's what seems to happen for INSAR_ISCE as noted here.

@cmarshak
Copy link
Contributor Author

I don't see it added to any deployments. @cmarshak did you want to add to hyp3-tibet-jpl-test, hyp3-tibet-jpl, and hyp3-nisar-jpl like OPERA_DIST_S1?

I put it in hyp3-tibet-jpl and the corresponding test deployment.

Thanks so much.

Just updated the schema (had DIST_S1 instead of DIST_S1_CONFIRMATION), reduced memory, and ensure we always pass at least two granules.

@cmarshak
Copy link
Contributor Author

Super appreciate your time @jhkennedy!

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

Comments