#81: assign streams to fixed variables#90
Conversation
|
NOTE: In order to make these variables available, an update will be required to source the full mappings json file rather than the condensed version originally used. This should be incorporated within #89 and special attention made to make sure key name changes between the 2 json files do not affect functionality. EDIT: It may instead be more valuable to inject the mapping information into the condensed json file currently used. This is far less complex than changing all other references and directly copying a much larger file when not absolutely necessary. |
matthew-mizielinski
left a comment
matthew-mizielinski
left a comment
There was a problem hiding this comment.
in principle ok, but I would like the afx/ofx stream sorted
| if realm in ["ocean", "ocnBgchem", "seaIce"]: | ||
| stream = "ofx" | ||
| elif realm in ["atmos", "aerosol", "atmosChem", "land", "landIce"]: | ||
| stream = "afx" |
There was a problem hiding this comment.
might need to confirm that this is correct; there could be some seaIce in the afx stream, but check with @mo-gill as to which way round they are
There was a problem hiding this comment.
I have cross referenced with Issue 915 on CDDS (Produce BCVs) and this all seems consistent.
2 participants
Closes issue #81