Skip to content

Feature/8505 index swot ssh images files#1273

Open
weited wants to merge 2 commits into
masterfrom
feature/8505-index-swot-ssh-images-files
Open

Feature/8505 index swot ssh images files#1273
weited wants to merge 2 commits into
masterfrom
feature/8505-index-swot-ssh-images-files

Conversation

@weited
Copy link
Copy Markdown
Contributor

@weited weited commented May 28, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 28, 2026 01:16
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds indexing support for SWOT SSH GIF images by registering a new swotGsla-ssh product config that scans DR_SWOT/SSH/<region>/*.gif with a whitelisted region list, plus fixtures and assertions verifying both inclusion of allowed regions and exclusion of the archive directory.

Changes:

  • Add swotGsla-ssh entry to FILE_PATH_CONFIG with a 3-layer include pattern and region whitelist.
  • Add test fixture GIFs under DR_SWOT/SSH/{Au,Tas,archive}/.
  • Extend integration test with expected output and an assertion that archive is not indexed.

Reviewed changes

Copilot reviewed 2 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
ARGO/oceancurrent/oceancurrent_file_server_api.py New swotGsla-ssh config indexing DR_SWOT/SSH/<region>/*.gif.
ARGO/oceancurrent/test_oceancurrent_file_server_api.py Adds expected JSON entries for SWOT SSH and asserts archive is excluded.
ARGO/oceancurrent/tests/.../DR_SWOT/SSH/Au/20260426223317.gif Fixture for whitelisted region Au.
ARGO/oceancurrent/tests/.../DR_SWOT/SSH/Tas/20260424190628.gif Fixture for whitelisted region Tas.
ARGO/oceancurrent/tests/.../DR_SWOT/SSH/archive/20260101000000.gif Fixture to verify archive exclusion.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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