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

Fixes ALM and RTM timers #496

Merged
merged 2 commits into from
Jan 14, 2016
Merged

Conversation

bishtgautam
Copy link
Contributor

Corrects the timers for ALM and RTM

Fixes #289 #483

[BFB]

@bishtgautam
Copy link
Contributor Author

@daliwang This PR has trivial code modifications and could be merged right away. This will allow @worleyph can continue to collect performance data on relatively inexpensive land+rtm components :).

Before this PR:

>cat cesm_timing_stats.151123-123909 | grep -i rtmi_grid
rtmi_grid                                      0        0 0.000000e+00   0.000000e+00     0.000 (     0      0)     0.000 (     0      0)

After this PR:

>cat cesm_timing_stats.151123-125937 | grep rtmi_grid
rtmi_grid                                    192      192 1.920000e+02   6.820641e+01     0.361 (   169      0)     0.346 (    23      0)

@bishtgautam
Copy link
Contributor Author

Self-integrating it.

@bishtgautam bishtgautam assigned bishtgautam and unassigned worleyph Jan 14, 2016
@worleyph
Copy link
Contributor

Thanks! Was waiting for the V1 churn to settle down, but I am always happy to give up an integration task :-).

bishtgautam pushed a commit that referenced this pull request Jan 14, 2016
Corrects the timers for ALM and RTM

Fixes #289 #483

[BFB]

Conflicts:
	components/rtm/src/riverroute/RtmMod.F90
@bishtgautam bishtgautam merged commit c8eca0d into master Jan 14, 2016
bishtgautam pushed a commit that referenced this pull request Jan 14, 2016
Corrects the timers for ALM and RTM

Fixes #289 #483

[BFB]

Conflicts:
	components/rtm/src/riverroute/RtmMod.F90
@bishtgautam bishtgautam deleted the bishtgautam/lnd/timer-bug-fixes branch January 23, 2016 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants