You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The situation at the moment, as I understand it, is a little messy:
ACCESS-OM2 uses the master branch which essentially adds some CI on top of mom-ocean/MOM5master
ACCESS-ESM1.5 uses the access-esm1.5 branch
Modifications to MOM5needed only in ACCESS-OM2 require:
PR to mom-ocean/MOM5master, review, merge
PR to update ACCESS-NRI/MOM5master, review, merge
Additional PRs to update ACCESS-OM2
Modifications to MOM5needed only in ACCESS-ESM1.5 require:
PR to ACCESS-NRI/MOM5access-esm1.5, review, merge
Additional PRs to update ACCESS-ESM1.5
Modifications to MOM5needed in bothACCESS-OM2 and ACCESS-ESM1.5 require:
PR to mom-ocean/MOM5master, review, merge
PR to update ACCESS-NRI/MOM5master, review, merge
equivalent PR to ACCESS-NRI/MOM5access-esm1.5, review, merge (in some cases this could be cherry-picked from master)
Additional PRs to update ACCESS-OM2 and ACCESS-ESM1.5
The need to open the equivalent PR to ACCESS-NRI/MOM5access-esm1.5 is time-consuming and error-prone.
So... do we really need an access-esm1.5 branch? MOM5 src is already riddled with compiler preprocessor directives for ACCESS_OM and ACCESS_CM. Could all three models be reconciled into a single codebase or is this too difficult/not worth it at this point?
The text was updated successfully, but these errors were encountered:
The situation at the moment, as I understand it, is a little messy:
ACCESS-OM2
uses themaster
branch which essentially adds some CI on top ofmom-ocean/MOM5
master
ACCESS-ESM1.5
uses theaccess-esm1.5
branchModifications to
MOM5
needed only inACCESS-OM2
require:mom-ocean/MOM5
master
, review, mergeACCESS-NRI/MOM5
master
, review, mergeACCESS-OM2
Modifications to
MOM5
needed only inACCESS-ESM1.5
require:ACCESS-NRI/MOM5
access-esm1.5
, review, mergeACCESS-ESM1.5
Modifications to
MOM5
needed in bothACCESS-OM2
andACCESS-ESM1.5
require:mom-ocean/MOM5
master
, review, mergeACCESS-NRI/MOM5
master
, review, mergeACCESS-NRI/MOM5
access-esm1.5
, review, merge (in some cases this could be cherry-picked frommaster
)ACCESS-OM2
andACCESS-ESM1.5
The need to open the equivalent PR to
ACCESS-NRI/MOM5
access-esm1.5
is time-consuming and error-prone.So... do we really need an
access-esm1.5
branch? MOM5 src is already riddled with compiler preprocessor directives forACCESS_OM
andACCESS_CM
. Could all three models be reconciled into a single codebase or is this too difficult/not worth it at this point?The text was updated successfully, but these errors were encountered: