Skip to content

Adding script for computing exf_albedo#28

Merged
spiani merged 3 commits into
mainfrom
adding_albedo
Jun 24, 2026
Merged

Adding script for computing exf_albedo#28
spiani merged 3 commits into
mainfrom
adding_albedo

Conversation

@spiani

@spiani spiani commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 24, 2026 09:07
@spiani spiani self-assigned this Jun 24, 2026

Copilot AI 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.

Pull request overview

This PR adds a new CLI subcommand to compute and write an EXF albedo value for a domain, and wires it into the mitgcm_inputs command dispatcher.

Changes:

  • Added exf_albedo subcommand implementation with CLI args for meshmask input and output file.
  • Registered the new subcommand in the main CLI (src/mitgcm_inputs/__main__.py).

Reviewed changes

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

File Description
src/mitgcm_inputs/exf_albedo/__init__.py New command implementation for computing/writing albedo based on domain latitude.
src/mitgcm_inputs/__main__.py Adds exf_albedo imports, subparser registration, and command dispatch mapping.

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

Comment thread src/mitgcm_inputs/exf_albedo/__init__.py
Comment thread src/mitgcm_inputs/exf_albedo/__init__.py
@spiani spiani requested a review from stefanoquerin June 24, 2026 09:11
@spiani spiani merged commit 79194af into main Jun 24, 2026
1 check passed
@spiani spiani deleted the adding_albedo branch June 24, 2026 12: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.

3 participants