Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add land region script and update Manifest to branch #757

Merged
merged 1 commit into from
Sep 12, 2024
Merged

add land region script and update Manifest to branch #757

merged 1 commit into from
Sep 12, 2024

Conversation

kmdeck
Copy link
Member

@kmdeck kmdeck commented Sep 5, 2024 •
edited
Loading

Purpose

Create a regional run example which runs when we run long runs.

In the future, we will move this to a tutorial.

To-do

Content

  • long lat swap in Plane constructor
  • regional land script
  • I have read and checked the items on the review checklist.

@kmdeck kmdeck marked this pull request as ready for review September 10, 2024 23:25
@kmdeck kmdeck requested a review from Sbozzolo September 11, 2024 00:23
Sbozzolo
Sbozzolo reviewed Sep 11, 2024
View reviewed changes
src/shared_utilities/Domains.jl Outdated Show resolved Hide resolved
experiments/long_runs/land_region.jl Outdated Show resolved Hide resolved
experiments/long_runs/land_region.jl Outdated Show resolved Hide resolved
experiments/long_runs/land_region.jl Outdated Show resolved Hide resolved
@kmdeck kmdeck requested a review from Sbozzolo September 12, 2024 00:14
Sbozzolo
Sbozzolo approved these changes Sep 12, 2024
View reviewed changes
experiments/long_runs/land_region.jl Outdated Show resolved Hide resolved
@kmdeck kmdeck enabled auto-merge September 12, 2024 16:04
@kmdeck kmdeck merged commit 6627c51 into main Sep 12, 2024
11 of 12 checks passed
@kmdeck kmdeck mentioned this pull request Sep 12, 2024
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sbozzolo Sbozzolo Sbozzolo approved these changes

Assignees
No one assigned
Labels
Run long runs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kmdeck @Sbozzolo