{"action":"created","issue":{"url":"https://api.github.com/repos/ESMCI/cime/issues/448","repository_url":"https://api.github.com/repos/ESMCI/cime","labels_url":"https://api.github.com/repos/ESMCI/cime/issues/448/labels{/name}","comments_url":"https://api.github.com/repos/ESMCI/cime/issues/448/comments","events_url":"https://api.github.com/repos/ESMCI/cime/issues/448/events","html_url":"https://github.com/ESMCI/cime/issues/448","id":172999404,"number":448,"title":"Convert ERP test to SystemTestsCompareTwo infrastructure?","user":{"login":"billsacks","id":6266741,"avatar_url":"https://avatars.githubusercontent.com/u/6266741?v=3","gravatar_id":"","url":"https://api.github.com/users/billsacks","html_url":"https://github.com/billsacks","followers_url":"https://api.github.com/users/billsacks/followers","following_url":"https://api.github.com/users/billsacks/following{/other_user}","gists_url":"https://api.github.com/users/billsacks/gists{/gist_id}","starred_url":"https://api.github.com/users/billsacks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/billsacks/subscriptions","organizations_url":"https://api.github.com/users/billsacks/orgs","repos_url":"https://api.github.com/users/billsacks/repos","events_url":"https://api.github.com/users/billsacks/events{/privacy}","received_events_url":"https://api.github.com/users/billsacks/received_events","type":"User","site_admin":false},"labels":[{"id":341451414,"url":"https://api.github.com/repos/ESMCI/cime/labels/ready","name":"ready","color":"ededed","default":false}],"state":"open","locked":false,"assignee":{"login":"mfdeakin-sandia","id":20424817,"avatar_url":"https://avatars.githubusercontent.com/u/20424817?v=3","gravatar_id":"","url":"https://api.github.com/users/mfdeakin-sandia","html_url":"https://github.com/mfdeakin-sandia","followers_url":"https://api.github.com/users/mfdeakin-sandia/followers","following_url":"https://api.github.com/users/mfdeakin-sandia/following{/other_user}","gists_url":"https://api.github.com/users/mfdeakin-sandia/gists{/gist_id}","starred_url":"https://api.github.com/users/mfdeakin-sandia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mfdeakin-sandia/subscriptions","organizations_url":"https://api.github.com/users/mfdeakin-sandia/orgs","repos_url":"https://api.github.com/users/mfdeakin-sandia/repos","events_url":"https://api.github.com/users/mfdeakin-sandia/events{/privacy}","received_events_url":"https://api.github.com/users/mfdeakin-sandia/received_events","type":"User","site_admin":false},"assignees":[{"login":"mfdeakin-sandia","id":20424817,"avatar_url":"https://avatars.githubusercontent.com/u/20424817?v=3","gravatar_id":"","url":"https://api.github.com/users/mfdeakin-sandia","html_url":"https://github.com/mfdeakin-sandia","followers_url":"https://api.github.com/users/mfdeakin-sandia/followers","following_url":"https://api.github.com/users/mfdeakin-sandia/following{/other_user}","gists_url":"https://api.github.com/users/mfdeakin-sandia/gists{/gist_id}","starred_url":"https://api.github.com/users/mfdeakin-sandia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mfdeakin-sandia/subscriptions","organizations_url":"https://api.github.com/users/mfdeakin-sandia/orgs","repos_url":"https://api.github.com/users/mfdeakin-sandia/repos","events_url":"https://api.github.com/users/mfdeakin-sandia/events{/privacy}","received_events_url":"https://api.github.com/users/mfdeakin-sandia/received_events","type":"User","site_admin":false}],"milestone":null,"comments":6,"created_at":"2016-08-24T16:28:21Z","updated_at":"2016-11-16T01:37:42Z","closed_at":null,"body":"We want to convert the various system tests that compare two runs to use the SystemTestsCompareTwo infrastructure, where possible. I am opening separate issues for each such system test, so that we can check them off as we do this conversion.\n\nThe ERP test would be less straightforward to convert to this infrastructure, since currently it makes use of setting CONTINUE_RUN which doesn't work if we're using a clone-based test. On the other hand, the criticality and complexity of the ERP test argue for translating it to this new infrastructure, which will make it more maintainable and less error-prone.\n\n@mvertens suggests changing its behavior so that, rather than doing a CONTINUE_RUN, it instead uses a branch run for the second case. For this to work, we'd need to introduce a new hook into the SystemTestsCompareTwo infrastructure: a _pre_run_two hook that is executed between the case1 run and the case2 run. For the ERP test, this would need to copy (or link) the relevant restart files from the case1 run directory to the case2 run directory.\n"},"comment":{"url":"https://api.github.com/repos/ESMCI/cime/issues/comments/260827386","html_url":"https://github.com/ESMCI/cime/issues/448#issuecomment-260827386","issue_url":"https://api.github.com/repos/ESMCI/cime/issues/448","id":260827386,"user":{"login":"jedwards4b","id":6126104,"avatar_url":"https://avatars.githubusercontent.com/u/6126104?v=3","gravatar_id":"","url":"https://api.github.com/users/jedwards4b","html_url":"https://github.com/jedwards4b","followers_url":"https://api.github.com/users/jedwards4b/followers","following_url":"https://api.github.com/users/jedwards4b/following{/other_user}","gists_url":"https://api.github.com/users/jedwards4b/gists{/gist_id}","starred_url":"https://api.github.com/users/jedwards4b/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jedwards4b/subscriptions","organizations_url":"https://api.github.com/users/jedwards4b/orgs","repos_url":"https://api.github.com/users/jedwards4b/repos","events_url":"https://api.github.com/users/jedwards4b/events{/privacy}","received_events_url":"https://api.github.com/users/jedwards4b/received_events","type":"User","site_admin":false},"created_at":"2016-11-16T01:37:42Z","updated_at":"2016-11-16T01:37:42Z","body":"system_tests_compare_two.py isn't calling run_indv with all the available arguments, i think that you just need to add that st_archive argument there to make it available to tests using that method.\r\n"}}
{"actor":{"display_login":"jedwards4b"}}