Last edited Timestamp?
For completion by the Sci/Tech/Code? reviewer
Reviewer: Rachid Benshila
Ticket Details, Documentation and Code changes
Do you understand the area of code being altered and the reasoning why it is being altered? | YES/NO |
Do the proposed code changes correspond with the stated reason for the change? | YES/NO |
Is the in-line documentation accurate and sufficient? | YES/NO |
Do the code changes comply with NEMO coding standards? | YES/NO |
Is the Ticket documented with sufficient detail for others to understand the impact of the change? | YES/NO |
Does any corresponding external documentation require updating? | YES/NO |
If yes, which docs and have the updates been drafted? | YES/NO |
Are namelist changes required for this change? | YES/NO |
If yes, have they been done? | YES/NO |
Has a completed Ticket Summary template been appended to the ticket to aid code reviews | YES/NO |
Does this summary correspond with your understanding of the full ticket? | YES/NO |
Ticket, Documentation and Code comments
Add specific Ticket, Documentation and code comments here
Testing
Has the NVTK and other jobs been tested with this change? | YES/NO |
Have the required bit comparability tests been run? | YES/NO |
Can this change be shown to have a null impact? (if option not selected) | YES/NO |
If no, is reason for the change valid/understood? | YES/NO/NA |
If no, ensure that the ticket details the impact this change will have on model configurations . | YES/NO/NA |
Is this change expected to preserve all diagnostics? | YES/NO |
If no, is reason for the change valid/understood? | YES/NO/NA |
Are there significant changes in run time/memory? | YES/NO |
Testing Comments
Add specific testing comments here
Add specific testing comments here
Code Review
Do the code changes comply with NEMO coding standards? | YES/NO |
Are code changes consistent with the design of NEMO? | YES/NO |
Is the code free of unwanted TABs? | YES/NO |
Has the code been wholly (100%) produced by NEMO developers working on NEMO? | YES/NO |
If no, ensure collaboration agreement has been added to the ticket keywords |
Add specific code comments or suggested alterations here.
Review Summary
Add summary here
Approval for the trunk
YES/NO
The code reviewer may approve the change for the NEMO trunk when:
- their requests/comments have been addressed satisfactorily.
- the above check-list has been completed.
or the code reviewer may choose to reject & assign the change back to the code author.
Last modified 14 years ago
Last modified on 2010-06-25T16:54:27+02:00