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

fix gpu longrun pipeline #3052

Merged
merged 1 commit into from
May 29, 2024
Merged

fix gpu longrun pipeline #3052

merged 1 commit into from
May 29, 2024

Conversation

szy21
Copy link
Member

@szy21 szy21 commented May 27, 2024

Purpose

Fixes a typo in longrun GPU pipeline. Also moves longrun_aquaplanet_rhoe_equil_55km_nz63_clearsky_progedmf_diffonly_0M to CPU as it doesn't work on GPU. Also updates climacommon.

To-do

Content


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

@szy21 szy21 enabled auto-merge May 27, 2024 15:14
@szy21 szy21 requested a review from Sbozzolo May 27, 2024 19:20
@Sbozzolo
Copy link
Member

If you merge this, you'll have to remember to update the climacommon module when it is released.

Comment on lines +73 to +74
slurm_ntasks: 64
slurm_nodes: 4
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will be much slower than choosing 56 tasks on 1 node

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I usually need to wait for very long (more than a few hours) if I ask for 1 node.

@szy21
Copy link
Member Author

szy21 commented May 27, 2024

If you merge this, you'll have to remember to update the climacommon module when it is released.

I can wait until the climacommon is released.

@szy21 szy21 added this pull request to the merge queue May 27, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 27, 2024
@szy21 szy21 enabled auto-merge May 29, 2024 01:49
@szy21 szy21 added this pull request to the merge queue May 29, 2024
Merged via the queue into main with commit 6635bd0 May 29, 2024
10 of 11 checks passed
@szy21 szy21 deleted the zs/hotfix branch May 29, 2024 03:50
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