Skip to content

Conversation

@jtruesdal
Copy link
Collaborator

@jtruesdal jtruesdal commented Dec 19, 2025

These changes to enable nonhydrostatic mode were provided by Timothy Andrews. In addition to Timothy's changes I modified the fv3_arrays.F90 file to allow the user to update the namelist variables rearth and omega to mimic the fv3 small_earth_scaling functionality. For example to enable a 1/20th scaling add omega and rearth to the user_nl_cam namelist with values set to omega * 20 and rearth / 20.

Copilot AI review requested due to automatic review settings December 19, 2025 06:30
@jtruesdal jtruesdal marked this pull request as draft December 19, 2025 06:30
Copy link

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.

Copilot reviewed 23 out of 26 changed files in this pull request and generated 3 comments.


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

@jtruesdal
Copy link
Collaborator Author

jtruesdal commented Dec 19, 2025

@ta440, I see that I forgot to close the loop on adding your fv3 mods. I created a new PR and was hoping that you might have time to test it and make sure that you see the tilt? Also you should be able to modify rearth and omega via the user_nl_cam namelist and create a small earth scaling by setting them directly in the user_nl_cam file to rearth/20. (=318561) and omega * 20 (=1.45842e-3). After you verify that this works I'll add a fv3_small_earth_scale namelist variable that will modify the default rearth and omega by some real value (default 20.)

@jtruesdal jtruesdal requested a review from Copilot December 19, 2025 17:52
Copy link

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

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.


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

@jtruesdal
Copy link
Collaborator Author

I pulled out the extraneous mods. All mods are now just enabling nonhydrostatic mode.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jtruesdal jtruesdal marked this pull request as ready for review December 19, 2025 18:03
Copilot AI review requested due to automatic review settings December 19, 2025 18:03
Copy link

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

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


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

@jtruesdal jtruesdal added the ready_for_ci This PR is ready for CI tests on CIRRUS cloud label Dec 19, 2025
@github-actions github-actions bot removed the ready_for_ci This PR is ready for CI tests on CIRRUS cloud label Dec 19, 2025
@github-actions
Copy link

CI Test Failed

I have removed the ready_for_ci label to prevent wasted resources.

Action Required:

  1. Check the GitHub Actions logs for errors.
  2. Fix the issues and push new commits.
  3. Re-apply the ready_for_ci label to restart the build.

@jtruesdal jtruesdal self-assigned this Dec 19, 2025
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