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

More efficient PE layout for NOINYOC compset #391

Merged

Conversation

JorgSchwinger
Copy link
Contributor

The current PE-layout for the NOINYOC compset on the tnx1v4 grid is very inefficient (16.56 simulated_years/day with hybrid coordinates on betzy). The new layout proposed here runs 27 years/day using the SAME number of nodes. This was achieved by increasing the PE-count for BLOM to 431 (new patch file created) and reducing the CICE PE-count to 80. Ice still runs much faster with the reduced PE-count. Results are bit-for-bit despite the changed number of CICE PEs.

Copy link
Contributor

@TomasTorsvik TomasTorsvik left a comment

Choose a reason for hiding this comment

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

@JorgSchwinger - thanks, looks fine to me.

Would it make sense to introduce the same PE layout for the a%TL319.+oi%tnx1v4 configuration?

  <grid name="a%TL319.+oi%tnx1v4">
    <mach name="any">
      <pes pesize="M" compset="_DATM.*_BLOM%ECO">
        <comment>none</comment>
        <ntasks>
          <ntasks_atm>128</ntasks_atm>
          <ntasks_rof>128</ntasks_rof>
          <ntasks_ice>128</ntasks_ice>
          <ntasks_ocn>354</ntasks_ocn>
          <ntasks_cpl>128</ntasks_cpl>
          <ntasks_lnd>128</ntasks_lnd>
          <ntasks_glc>128</ntasks_glc>
          <ntasks_wav>128</ntasks_wav>
        </ntasks>

@JorgSchwinger
Copy link
Contributor Author

This would definitely make sense, yes. I have not tested it yet though. So I was hesitant to modify it without testing. I suggest I will do this in a separate PR later.

@JorgSchwinger JorgSchwinger force-pushed the feature-newpes_NOINYOC branch from 5b23a72 to 23b9db2 Compare October 28, 2024 15:41
@JorgSchwinger JorgSchwinger merged commit f2b96c1 into NorESMhub:master Oct 28, 2024
5 checks passed
@JorgSchwinger JorgSchwinger deleted the feature-newpes_NOINYOC branch October 28, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants