{"action":"created","issue":{"url":"https://api.github.com/repos/StrikeNP/trac_test/issues/696","repository_url":"https://api.github.com/repos/StrikeNP/trac_test","labels_url":"https://api.github.com/repos/StrikeNP/trac_test/issues/696/labels{/name}","comments_url":"https://api.github.com/repos/StrikeNP/trac_test/issues/696/comments","events_url":"https://api.github.com/repos/StrikeNP/trac_test/issues/696/events","html_url":"https://github.com/StrikeNP/trac_test/issues/696","id":332199314,"node_id":"MDU6SXNzdWUzMzIxOTkzMTQ=","number":696,"title":"Create option to do linear interpolation in CLUBB instead of cubic monotonic spline interpolation (Trac #696)","user":{"login":"StrikeNP","id":2577059,"node_id":"MDQ6VXNlcjI1NzcwNTk=","avatar_url":"https://avatars0.githubusercontent.com/u/2577059?v=4","gravatar_id":"","url":"https://api.github.com/users/StrikeNP","html_url":"https://github.com/StrikeNP","followers_url":"https://api.github.com/users/StrikeNP/followers","following_url":"https://api.github.com/users/StrikeNP/following{/other_user}","gists_url":"https://api.github.com/users/StrikeNP/gists{/gist_id}","starred_url":"https://api.github.com/users/StrikeNP/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/StrikeNP/subscriptions","organizations_url":"https://api.github.com/users/StrikeNP/orgs","repos_url":"https://api.github.com/users/StrikeNP/repos","events_url":"https://api.github.com/users/StrikeNP/events{/privacy}","received_events_url":"https://api.github.com/users/StrikeNP/received_events","type":"User","site_admin":false},"labels":[{"id":963997231,"node_id":"MDU6TGFiZWw5NjM5OTcyMzE=","url":"https://api.github.com/repos/StrikeNP/trac_test/labels/Migrated%20from%20Trac","name":"Migrated from Trac","color":"FFFFFF","default":false},{"id":963997516,"node_id":"MDU6TGFiZWw5NjM5OTc1MTY=","url":"https://api.github.com/repos/StrikeNP/trac_test/labels/clubb_src","name":"clubb_src","color":"FFFFFF","default":false},{"id":963997043,"node_id":"MDU6TGFiZWw5NjM5OTcwNDM=","url":"https://api.github.com/repos/StrikeNP/trac_test/labels/enhancement","name":"enhancement","color":"a2eeef","default":true},{"id":964009148,"node_id":"MDU6TGFiZWw5NjQwMDkxNDg=","url":"https://api.github.com/repos/StrikeNP/trac_test/labels/schemena@uwm.edu","name":"schemena@uwm.edu","color":"FFFFFF","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":13,"created_at":"2018-06-13T23:22:10Z","updated_at":"2018-06-15T23:44:59Z","closed_at":null,"author_association":"OWNER","body":"\n**Introduction**\n\nIn ticket:461, we replaced linear interpolation in the vertical with cubic monotonic spline interpolation.  However, the only improvement was that a cloud formed in cgils_s12 (https://github.com/larson-group/clubb/issues/461).  Furthermore, the cost of cubic interpolation is 17% of the cost of CLUBB, exclusive of microphysics, radiation, and writing to disk (see timing output in ticket:694).  If linear interpolation costs half of cubic interpolation, we might be able to save 8% of the cost without much degradation in the results.  At the least, it would be nice to be able to switch back and forth between linear and cubic interpolation just by changing a flag.  Then we could ask NCAR, e.g., to test the two types of interpolation in global simulations.\n\n**Technical spec**\n\nWe want to re-introduce a flag, l_cubic_interp, and switch from cubic to linear interpolation just by switching the flag.  A key commit where cubic interpolation was introduced is r5457; another commit where it was made the default is r5472.  We want to generalize this code.  I think that we want zm2zt and zt2zm to be overloaded operators that change meaning depending on the value of l_cubic_interp.\n\nAlso, we should add idiot-proofing that stops the run with an error message if l_quintic_poly_interp = .true. and l_cubic_interp = .false.\n\n**Tests**\n\n * Results should be bindiff identical when l_cubic_interp = .true.\n\n * Differences in results should look similar to those shown in  https://github.com/larson-group/clubb/issues/461.\n\n\nAttachments:\n\n[Cubic_Linear_plots.maff](https://github.com/larson-group/trac_attachment_archive/blob/master/clubb/Cubic_Linear_plots.maff)\n[Cubic_timing.er.tar.gz](https://github.com/larson-group/trac_attachment_archive/blob/master/clubb/Cubic_timing.er.tar.gz)\n[Linear_timing.er.tar.gz](https://github.com/larson-group/trac_attachment_archive/blob/master/clubb/Linear_timing.er.tar.gz)\n[Cubic_Linear_plots_v2.maff](https://github.com/larson-group/trac_attachment_archive/blob/master/clubb/Cubic_Linear_plots_v2.maff)\n[Cubic_timing_with_ARM_v2.er.tar.gz](https://github.com/larson-group/trac_attachment_archive/blob/master/clubb/Cubic_timing_with_ARM_v2.er.tar.gz)\n[Linear_timing_with_ARM_v2.er.tar.gz](https://github.com/larson-group/trac_attachment_archive/blob/master/clubb/Linear_timing_with_ARM_v2.er.tar.gz)\n[Grid_Class_Call_Structure.pdf](https://github.com/larson-group/trac_attachment_archive/blob/master/clubb/Grid_Class_Call_Structure.pdf)\n\nMigrated from http://carson.math.uwm.edu/trac/clubb/ticket/696\n```json\n{\n    \"status\": \"closed\", \n    \"changetime\": \"2014-06-16T12:54:40\", \n    \"description\": \"\\n'''Introduction'''\\n\\nIn ticket:461, we replaced linear interpolation in the vertical with cubic monotonic spline interpolation.  However, the only improvement was that a cloud formed in cgils_s12 (comment:24:ticket:461).  Furthermore, the cost of cubic interpolation is 17% of the cost of CLUBB, exclusive of microphysics, radiation, and writing to disk (see timing output in ticket:694).  If linear interpolation costs half of cubic interpolation, we might be able to save 8% of the cost without much degradation in the results.  At the least, it would be nice to be able to switch back and forth between linear and cubic interpolation just by changing a flag.  Then we could ask NCAR, e.g., to test the two types of interpolation in global simulations.\\n\\n'''Technical spec'''\\n\\nWe want to re-introduce a flag, l_cubic_interp, and switch from cubic to linear interpolation just by switching the flag.  A key commit where cubic interpolation was introduced is r5457; another commit where it was made the default is r5472.  We want to generalize this code.  I think that we want zm2zt and zt2zm to be overloaded operators that change meaning depending on the value of l_cubic_interp.\\n\\nAlso, we should add idiot-proofing that stops the run with an error message if l_quintic_poly_interp = .true. and l_cubic_interp = .false.\\n\\n'''Tests'''\\n\\n * Results should be bindiff identical when l_cubic_interp = .true.\\n\\n * Differences in results should look similar to those shown in  comment:24:ticket:461.\\n\", \n    \"reporter\": \"vlarson@uwm.edu\", \n    \"cc\": \"vlarson@uwm.edu\", \n    \"resolution\": \"fixed\", \n    \"_ts\": \"1402923280631605\", \n    \"component\": \"clubb_src\", \n    \"summary\": \"Create option to do linear interpolation in CLUBB instead of cubic monotonic spline interpolation\", \n    \"priority\": \"major\", \n    \"keywords\": \"\", \n    \"time\": \"2014-06-10T17:26:51\", \n    \"milestone\": \"\", \n    \"owner\": \"schemena@uwm.edu\", \n    \"type\": \"enhancement\"\n}\n```\n"},"comment":{"url":"https://api.github.com/repos/StrikeNP/trac_test/issues/comments/397768301","html_url":"https://github.com/StrikeNP/trac_test/issues/696#issuecomment-397768301","issue_url":"https://api.github.com/repos/StrikeNP/trac_test/issues/696","id":397768301,"node_id":"MDEyOklzc3VlQ29tbWVudDM5Nzc2ODMwMQ==","user":{"login":"StrikeNP","id":2577059,"node_id":"MDQ6VXNlcjI1NzcwNTk=","avatar_url":"https://avatars0.githubusercontent.com/u/2577059?v=4","gravatar_id":"","url":"https://api.github.com/users/StrikeNP","html_url":"https://github.com/StrikeNP","followers_url":"https://api.github.com/users/StrikeNP/followers","following_url":"https://api.github.com/users/StrikeNP/following{/other_user}","gists_url":"https://api.github.com/users/StrikeNP/gists{/gist_id}","starred_url":"https://api.github.com/users/StrikeNP/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/StrikeNP/subscriptions","organizations_url":"https://api.github.com/users/StrikeNP/orgs","repos_url":"https://api.github.com/users/StrikeNP/repos","events_url":"https://api.github.com/users/StrikeNP/events{/privacy}","received_events_url":"https://api.github.com/users/StrikeNP/received_events","type":"User","site_admin":false},"created_at":"2018-06-15T23:44:59Z","updated_at":"2018-06-15T23:44:59Z","author_association":"OWNER","body":"**Trac update on 2014/06/12 at 20:34**\nschemena@uwm.edu changed attachment from \"\" to \"Cubic_timing_with_ARM_v2.er.tar.gz\""}}
{"actor":{"display_login":"StrikeNP"}}