Skip to content

Fix: rename sixDaySst-centile productId to sixDaySst-centiles#1271

Merged
shaunahu merged 1 commit into
masterfrom
fix/oceancurrent-centile-product-id
Feb 18, 2026
Merged

Fix: rename sixDaySst-centile productId to sixDaySst-centiles#1271
shaunahu merged 1 commit into
masterfrom
fix/oceancurrent-centile-product-id

Conversation

@weited

@weited weited commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects a naming inconsistency in the oceancurrent file server API by renaming the productId from "sixDaySst-centile" (singular) to "sixDaySst-centiles" (plural). This aligns with the established naming convention used by other centile-related products in the codebase, such as "adjustedSeaLevelAnomaly-centiles".

Changes:

  • Updated two productId entries in the FILE_PATH_CONFIG from "sixDaySst-centile" to "sixDaySst-centiles"
  • Affects configurations for both DR_SST_daily and STATE_daily paths with pctiles layer

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

Comment thread ARGO/oceancurrent/oceancurrent_file_server_api.py
shaunahu
shaunahu previously approved these changes Feb 18, 2026

@shaunahu shaunahu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

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


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

@shaunahu shaunahu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@shaunahu
shaunahu merged commit 2e4d062 into master Feb 18, 2026
6 of 8 checks passed
@shaunahu
shaunahu deleted the fix/oceancurrent-centile-product-id branch February 18, 2026 05:01
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.

3 participants