Skip to content

Rename _is_agrid to _is_cgrid - #3059

Open
hayatexe wants to merge 1 commit into
Parcels-code:mainfrom
hayatexe:rename-is-agrid
Open

hayatexe wants to merge 1 commit into
Parcels-code:mainfrom
hayatexe:rename-is-agrid

Conversation

@hayatexe

Copy link
Copy Markdown

Description

Renamed _is_agrid to _is_cgrid and inverted its logic as suggested by the maintainer, since the original function also returned True for B-grid data. This change does not affect the program's behavior.

Checklist

  • Closes Rename _is_agrid #2692
  • This PR targets the correct branch (main for normal development, v3-support for v3 support)

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested any AI-generated content in my PR.
    • I take responsibility for any AI-generated content in my PR.
    • Describe how you used it (e.g., by pasting your prompt): An AI assistant made the rename, I reviewed and tested it.

The check (U and V defined on the same dimensions) also holds for
B-grids, so the old name was misleading. Rename to _is_cgrid and
negate the logic. No behaviour change.

Fixes Parcels-code#2692
Copilot AI lite review requested due to automatic review settings November 13, 2026 10:04

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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

This branch has not been deployed

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

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

Rename _is_agrid

2 participants