Follow instructions below to proceed with the new ACME code feature This is a main page to fulfill and capture the Code Review documentation on all answer changing code features that you want to include in E3SM model code base. Before such a feature can be integrated into the code, it needs to go through the Code Review process that is defined in /wiki/spaces/CNCL/pages/25231511 page and implemented here.
Tip |
---|
Follow this steps to complete the Code Review Process for your answer changing code before introducing it into E3SM code base: Expand |
---|
- Give your feature a
|
|
...
- name that includes the component and short description:
- Use a format [ W, B, C, P,
|
|
...
...
- that starts with a capital letter of your Group followed by a number (unique consecutive number for your group), followed by underscore then
|
|
...
...
...
...
...
...
...
- W1_UA_Surface_Flux
- Create a main Confluence Page to contain all the documentation on your the new feature to be integrated into
|
|
...
...
...
...
- or click on →
Create from Template |
---|
templateName | 976519950 |
---|
templateId | 976519950 |
---|
buttonLabel | Create "E3SM Code Review" page |
---|
|
- Title it with the name of your feature. This page contains two tables, fill them in as described in the template
- A feature
|
|
...
- dependency table – info from this table will feed the New Features Dependencies Overview page, that will help display the dependencies between feature pull requests
- A table with the checklist of all stages of the review and to obtain approvals for each of the stage of the Code Review
As you work on integrating your feature into E3SM code, step along the Checklist table on your main page, obtain the approvals before going to the next step
- Click on your new feature Code Review main page (created in #2), and as you go through the checklist of the code review process (second table #2.b) create and fill the appropriate pages under the main page:
- Design Document Confluence page
|
|
...
...
- E3SM Design Document' or click on →
Create from Template |
---|
templateName | 976520002 |
---|
templateId | 976520002 |
---|
buttonLabel | Create "E3SM Design Document" page |
---|
|
- Verification , Validation and Performance Phase 1 pages with appropriate templates: 'E3SM Code Verification Phase 1' , etc : →
Create from Template |
---|
templateName | 976552560 |
---|
templateId | 976552560 |
---|
buttonLabel | Create "E3SM Code Verification Phase 1" page |
---|
| Create from Template |
---|
templateName | 976552566 |
---|
templateId | 976552566 |
---|
buttonLabel | Create "ACME Code Performance Phase 1" page |
---|
| Create from Template |
---|
templateName | 976552572 |
---|
templateId | 976552572 |
---|
buttonLabel | Create "ACME Code Validation Phase 1" page |
---|
|
- Phase 2 Plan - use '
|
|
...
- E3SM Code Review Phase 2 Plan' (TBD)
- Coupled Performance and Coupled Validation pages use '
|
|
...
- E3SM Feature Coupled Performance ' or'
|
|
...
- E3SM Feature Coupled Validation'templates
|
|
...
- (TBD)
- When the Phase 1 documentation checklist (table #2.b) is satisfied and you have the approval from a Group Lead (the blue section of the table), you are ready to issue a PR and assign it to an integrator
|
|
...
See example documentation in O1_Template-1 example feature documentation.
Page Tree |
---|
root | Code Review Process |
---|
excerpt | true |
---|
|
Code Features
...
- . At that point fill in the blue section in the dependencies table (table #2.a) with the name of the PR integrator's name, and after he integrates the code, provide the integration date.
- Phase 2 starts with the Plan document (#3.c). The coupled system integration and testing can and should be done with the aggregate of features, and the planned coupled simulations may validate a combination of feature aggregates if feasible.
Instead of creating your own page as in description, you can use template pages already created for you. Use any pages with name template in them in the Proposed New Features - simply change the template pages name to your feature name and fill in the pages. |
For a good examaple, check this competed documentation: A1 Ozone Hole (Linoz-v2) |
New Feature Dependencies, PR Dates, Git Branch Overview
To check dependencies between new features, what blocks them and what do they affect, the dates they are ready for Pull Request (PR), their git branch, date PR was issued, who it was assigned to, when it was integrated into E3SM and the associated E3SM git hash/tag (remember the table is sortable):
Design Document Overview
For the quick overview of all design documents, which part is already done, if it is ready for GLs approval, please see:
...
Note |
---|
|
V2 Code Features Main Pages Page Properties Report |
---|
firstcolumn | Feature Main Page |
---|
spaces | @all |
---|
headings | 2.Owner, 6.Issued PR, 8.Integrated,9.Hash/Tag |
---|
pageSize | 100 |
---|
label | y1q3tr |
---|
cql | label = "code_rev_process" and ancestor = "36176475" |
---|
|
|
Note |
---|
|
V2 Features Design Documents Page Properties Report |
---|
firstcolumn | Design Document |
---|
spaces | @all |
---|
headings | 2.Owner, 3.Created, 4.Equ, 5.Ver, 6.Perf, 7.Val, 8.Approver, 9.Approved Date, V2.0 |
---|
pageSize | 100 |
---|
label | y1q3tr |
---|
cql | label = "design_doc" and ancestor = "36176475" |
---|
|
|
...
Warning |
---|
|
V1 Declined Code Features Main Pages Page Properties Report |
---|
firstcolumn | Feature Main Page |
---|
spaces | @all |
---|
headings | 2.Owner, 6.Issued PR, 8.Integrated,9.Hash/Tag |
---|
pageSize | 100 |
---|
label | y1q3tr |
---|
cql | label = "code_rev_process" and ancestor = "36176472" |
---|
|
|
...
Tip |
---|
|
V1 Code Features Main Pages Page Properties Report |
---|
firstcolumn | Feature Main Page |
---|
spaces | @all |
---|
headings | 2.Owner, 6.Issued PR, 8.Integrated |
---|
pageSize | 100 |
---|
label | y1q3tr |
---|
cql | label = " |
---|
|
|
...
...
and ancestor = "36176459" |
|
|
...
Children Pages Listing
Page Tree |
---|
root | Code Review Process Implementation |
---|
|