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

Improve YATM performance #8

Open
nichannah opened this issue Oct 24, 2018 · 0 comments
Open

Improve YATM performance #8

nichannah opened this issue Oct 24, 2018 · 0 comments
Assignees

Comments

@nichannah
Copy link
Contributor

nichannah commented Oct 24, 2018

There are a couple of ways that YATM performance could be improved:

  1. The runoff remapping checks conservation after applying weights to move from one grid to another and then again after moving all runoff that has landed to the ocean. Although it helps to do this twice if something goes wrong in one step but not the other the first check is redundant and it would make sense to remove this.

  2. Russ Fiedler [12:53 PM]
    @nic I think the remapping is very slow because you are checking all the ocean cells in the world. This makes the tree much larger than it needs to be too. Given that the runoff is coming from a 1.25 degree grid (?) you can mask the ocean points to be near the coast via if (any(land_sea_mask(i-halo:i+halo,j-halo:j+halo) < 0.5 and. land_sea_mask(i,j) > 0.5) then... where halo= 6 or so and is adjusted for boundaries.

  3. Russ Fiedler [1:59 PM]
    @nic How about hoisting the runoff remapping out of the loop starting at 113 and do it before entering? If you calculate the runoff ahead of time you don't need to synchronise with the fields that are passed earlier.

  4. read all the forcing fields before doing any OASIS puts. Keeping in mind that the puts are asynchronous so it should not be slowing things down as long as the ice is not waiting.

@nichannah nichannah changed the title Runoff remapping doing unnecessary conservation check Improve runoff remapping performance Dec 12, 2018
@nichannah nichannah changed the title Improve runoff remapping performance Improve YATM performance Dec 12, 2018
@nichannah nichannah self-assigned this Aug 5, 2019
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

No branches or pull requests

1 participant