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.

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

Add current target to longruns #354

Merged
merged 1 commit into from
Jul 27, 2023
Merged

Add current target to longruns #354

merged 1 commit into from
Jul 27, 2023

Conversation

LenkaNovak
Copy link
Collaborator

@LenkaNovak LenkaNovak commented Jul 21, 2023

Closes #355

ClimaCoupler-Longruns

Note: the old AMIP FINE benchmark crashes (unconverged SFs) after 120d. We will need to find the new stability region for these options, so it is commented for now.


  • I have read and checked the items on the review checklist.

@LenkaNovak LenkaNovak force-pushed the ln/revive-longruns branch 4 times, most recently from 317ce52 to 01ac7ed Compare July 25, 2023 14:42
@LenkaNovak LenkaNovak marked this pull request as ready for review July 25, 2023 16:33
@szy21
Copy link
Member

szy21 commented Jul 25, 2023

The MPI AMIP FINE: target longrun job looks good to me, thanks!

@LenkaNovak LenkaNovak force-pushed the ln/revive-longruns branch 3 times, most recently from 205e685 to cde6c93 Compare July 26, 2023 15:04
@LenkaNovak
Copy link
Collaborator Author

bors r+

bors bot added a commit that referenced this pull request Jul 26, 2023
354: Add current target to longruns r=LenkaNovak a=LenkaNovak



Co-authored-by: LenkaNovak <lenka@caltech.edu>
@bors
Copy link
Contributor

bors bot commented Jul 26, 2023

Build failed:

@LenkaNovak
Copy link
Collaborator Author

bors r+

bors bot added a commit that referenced this pull request Jul 26, 2023
354: Add current target to longruns r=LenkaNovak a=LenkaNovak



Co-authored-by: LenkaNovak <lenka@caltech.edu>
@bors
Copy link
Contributor

bors bot commented Jul 26, 2023

Build failed:

@valeriabarra
Copy link
Member

FYI: When jobs fail with Error: The command exited with status 137, it is better to add memory, e.g. slurm_mem: 20GB than just retrying (which is time consuming)

@LenkaNovak
Copy link
Collaborator Author

Yeah, I was trying to avoid that but I think I will have to resort to that now, lol. Thanks for keeping an eye! 👀

longer slabplanet

fix InexactError-DiffEq won't allow dt_save=Inf

helem 6 test

sponge helem 6 test

sponge helem 6 test

bring back all long runs

modular fix

modular fix

wait

clean up

reduce unthreaded

rm breaking AMIP FINE run

revs

rev

target for n=32

all jobs max walltime ~6h

all buildkites 20GB
@LenkaNovak
Copy link
Collaborator Author

bors r+

@bors
Copy link
Contributor

bors bot commented Jul 27, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit ebf1f18 into main Jul 27, 2023
10 checks passed
@bors bors bot deleted the ln/revive-longruns branch July 27, 2023 05:48
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.

Update Longruns
4 participants