search

E3SM-Project / transport_se

Atmosphere transport mini app
1 stars 1 forks source link

Why two test cases (DCMIP 1 and DCMP 2)? #3

Closed mt5555 closed 9 years ago

mt5555 commented 9 years ago

This mini-app will mostly be used for performance work, and the test cases are needed to verify correctness. Is one test sufficient to verify correctness? When should a user need to run both tests?

halldm2000 commented 9 years ago

Verifying operational correctness and measuring performance strike me as two very different things. Perhaps there should be separate scripts for each purpose? Actually DCMIP test 3 should be added to verify correctness, as the two existing tests don't cover orography.

mt5555 commented 9 years ago

I dont think the tracers see orography - there should be no reference to it in the mini-app, except as it might influence the winds. So I think we can skip that one for the mini-app. (but it would be great to get it into standalone HOMME if it is not already).

What about we use the ne8 script just for quick turnaound, the ne30 script for verifying operational correctness, and shorten the simulation time in the ne120 script (and increase the number of tracers) and use that for performance testing?

mt5555 commented 9 years ago

https://github.com/ACME-Climate/transport_se/commit/87046e624cc3d085694b8caade269ee61ee2a6c8

easterrc commented 9 years ago

Hi Mark,

Others have probably informed you of this, but I’ve gotten about 10 emails like this today and a large number on June 26.

-- Dick Easter

Richard C. Easter Scientist Atmospheric Sciences and Global Change Division Pacific Northwest National Laboratory 902 Battelle Boulevard P.O. Box 999, MSIN K9-30 Richland, WA 99352 USA Tel: 509-372-6175 Fax: 509-372-6168 richard.easter@pnl.gov www.pnl.gov

From: Mark Taylor [mailto:notifications@github.com] Sent: Sunday, June 28, 2015 10:52 AM To: ACME-Climate/transport_se Subject: Re: [transport_se] Why two test cases (DCMIP 1 and DCMP 2)? (#3)

Closed #3https://github.com/ACME-Climate/transport_se/issues/3.

— Reply to this email directly or view it on GitHubhttps://github.com/ACME-Climate/transport_se/issues/3#event-342352296.

gold2718 commented 9 years ago

@easterrc,

By default, you are 'watching' this repository. To stop the emails, click on the notification icon (to the left of your profile icon at the top-right of your github window), select the 'Watching' tab, and 'Unwatch' any repositories for which you do not want email messages. There is also a 'Change notification settings' button on that page which allows you to not be automatically signed up for email for new repositories.