Skip to content

Commit d5c1fbd

Browse files
committed
add info about how to bring things back to CAM
1 parent 09bb504 commit d5c1fbd

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

docs/conversion/walkthrough.md

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -399,7 +399,7 @@ Bug discovers that there are no `pbuf` variables in lunar_tides! Yay!
399399
There are no variables that Bug needs to mark as initialized. At least, so he thinks. He may revisit this!
400400

401401
### 1j - Initial standard name check
402-
Bug does an initial check of the [standard names spreadsheet](https://docs.google.com/spreadsheets/d/1vpQ_xDZk00Z-_3SpW5N2EF3_FY6K7opNN4cqtSMlbwU/edit?gid=0#gid=0). He is able to find standard anmes for all of the variables except the calendar type, current date and julian date info. He adds those to the spreadsheet and sends an email to the other CAM SEs about needing to decide on a standard name for these variables.
402+
Bug does an initial check of the [standard names spreadsheet](https://docs.google.com/spreadsheets/d/1vpQ_xDZk00Z-_3SpW5N2EF3_FY6K7opNN4cqtSMlbwU/edit?gid=0#gid=0). He is able to find standard names for all of the variables except the calendar type, current date and julian date info. He adds those to the spreadsheet and sends an email to the other CAM SEs about needing to decide on a standard name for these variables.
403403

404404
### 1k - Update CAM interface call(s)
405405
Bug updates the call in `physpkg.F90`:
@@ -680,7 +680,12 @@ ncdata_check = '/glade/campaign/cesm/community/amwg/sima_baselines/cam_sima_test
680680
Bug gets extremely lucky (or maybe he's just really skilled) and there are no differences found in the atm.log file!
681681

682682
## 9 - Bring back into CAM
683-
Bug moves the CCPP-ized version of lunar_tides.F90 into the CAM source tree, updates `configure`, and runs CAM. He confirms that it runs and answers haven't changed.
683+
684+
1. Bug moves the CCPP-ized version of lunar_tides.F90 into the CAM source tree by checking out his branch's hash of the atmospheric_physics branch (NOTE: this will need to be a tag before the PR to CAM can be brought in).
685+
1. He renames the old `src/physics/cam/lunar_tides.F90` file to `src/physics/cam/lunar_tides_cam.F90`
686+
- He removes everything from `lunar_tides_cam.F90` except `lunar_tides_readnl`
687+
1. Bug updates the `lunar_tides_init` and `lunar_tides_tend` subroutine calls in `physpkg.F90` to reflect the new CCPP-ized versions (full calling list, changing `lunar_tides_tend` to `lunar_tides_run`)
688+
1. Bug updates `configure`, and runs CAM. He confirms that it runs and answers haven't changed.
684689

685690
Bug commits his changes to his fork/branch of the three repos: CAM, CAM-SIMA, atmospheric_physics.
686691

0 commit comments

Comments
 (0)