Feedback from the CMIP6 example about cmor

[dnadeau4]

I will get to them next week. Good suggestions. I hate the color theme as well.... Don't know if I have time to get to them though....

[ehogan]

Another quick comment; in https://github.com/PCMDI/cmor/blob/master/Test/test_python_CMIP6_experimentID.json#L49, the institution_id is PCMDI, yet there is no entry for PCMDI in the institution_ids in https://github.com/PCMDI/cmor/blob/master/TestTables/CMIP6_CV.json#L120 and CMOR picks BNU for some reason. In fact, even if I use 'MOHC' as the institution_id and the correct string for institution, it still says Your input attribute institution "Met Office Hadley Centre, Fitzroy Road, Exeter, Devon, EX1 3PB, UK." will be replaced with "GCESS,BNU,Beijing,China" as defined in your Control Vocabulary file.

[dnadeau4]

Emma,

This test is to make sure that if you have a bad institution, it will be
replace by the right one. In
https://github.com/PCMDI/cmor/blob/master/Test/test_python_CMIP6_experimentID.json#L49
I have 2 institution_id, the second one "BNU" overwrite the first one.
Sorry about that.

Denis
On 6/20/16 4:40 AM, Emma Hogan wrote:

Another quick comment; in
https://github.com/PCMDI/cmor/blob/master/Test/test_python_CMIP6_experimentID.json#L49,
the |institution_id| is |PCMDI|, yet there is no entry for |PCMDI| in
the |institution_ids| in
https://github.com/PCMDI/cmor/blob/master/TestTables/CMIP6_CV.json#L120
and CMOR picks |BNU| for some reason. In fact, even if I use 'MOHC' as
the |institution_id| and the correct string for |institution|, it
still says |Your input attribute institution "Met Office Hadley
Centre, Fitzroy Road, Exeter, Devon, EX1 3PB, UK." will be replaced
with "GCESS,BNU,Beijing,China" as defined in your Control Vocabulary
file.|

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#69 (comment), or
mute the thread
https://github.com/notifications/unsubscribe/AGnMgmY9YMIjcA5AzXVP7CugyGI8rc7Yks5qNnxEgaJpZM4I3aOh.

[dnadeau4]

Actually, this test is to make sure that user institutionID is in the CV file. If not you get a critical error with an explanation to the. I need to replace the "???" as contact though. 👍

[dnadeau4]

I replaced the "???" character and checked in cmor-master.

[ehogan]

Thank you! Would it be possible to reopen this ticket? The items in my first comment haven't yet been dealt with :)

[dnadeau4]

Right now, I really really need to work on the CMIP6 Validator for ESGF. Many people are waiting for it. If you would be so inclined to create a pull request for me, I will merge it. Thanks!

[ehogan]

I just tried running the CMIP6 example using CMOR 3.1.1 and it completes without errors :)

Thank you for adding the comments to the CMIP6 example!

I think the only outstanding item is:

2. When loading the MIP table, should the line read table_id = cmor.load_table("CMIP6_Omon.json") (I was shown a return value when I ran the command in the example)?

I'm happy to submit a pull request if you think it is worthwhile, otherwise this issue can be closed.

Thank you! :)

[dnadeau4]

@ehogan I think this ticket can be closed, we should open a new one with anything pending. It seems that I deleted the "printf" debugging return value from cmor.load_table.