{"action":"created","issue":{"url":"https://api.github.com/repos/lmfit/lmfit-py/issues/287","labels_url":"https://api.github.com/repos/lmfit/lmfit-py/issues/287/labels{/name}","comments_url":"https://api.github.com/repos/lmfit/lmfit-py/issues/287/comments","events_url":"https://api.github.com/repos/lmfit/lmfit-py/issues/287/events","html_url":"https://github.com/lmfit/lmfit-py/pull/287","id":123594574,"number":287,"title":"ENH: Streamline Parameter._getval","user":{"login":"andyfaff","id":702934,"avatar_url":"https://avatars.githubusercontent.com/u/702934?v=3","gravatar_id":"","url":"https://api.github.com/users/andyfaff","html_url":"https://github.com/andyfaff","followers_url":"https://api.github.com/users/andyfaff/followers","following_url":"https://api.github.com/users/andyfaff/following{/other_user}","gists_url":"https://api.github.com/users/andyfaff/gists{/gist_id}","starred_url":"https://api.github.com/users/andyfaff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andyfaff/subscriptions","organizations_url":"https://api.github.com/users/andyfaff/orgs","repos_url":"https://api.github.com/users/andyfaff/repos","events_url":"https://api.github.com/users/andyfaff/events{/privacy}","received_events_url":"https://api.github.com/users/andyfaff/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-23T02:53:41Z","updated_at":"2015-12-23T03:14:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/lmfit/lmfit-py/pulls/287","html_url":"https://github.com/lmfit/lmfit-py/pull/287","diff_url":"https://github.com/lmfit/lmfit-py/pull/287.diff","patch_url":"https://github.com/lmfit/lmfit-py/pull/287.patch"},"body":"These changes speed up the `Parameter.value` attribute retrieval by a significant amount.\r\n\r\nThe following gists \r\n(unstreamlined _getval)\r\nhttps://gist.github.com/andyfaff/a0158e26e663b9308899\r\nand\r\n(streamlined _getval)\r\nhttps://gist.github.com/andyfaff/a3f4fc311a2c97c630fa\r\n\r\noutline that the overhead from _getval is reasonably large. These modifications halve the time required to evaluate the `userfcn` in this example. A total speedup of ~31% should be achievable in Minimizer.__residual."},"comment":{"url":"https://api.github.com/repos/lmfit/lmfit-py/issues/comments/166794721","html_url":"https://github.com/lmfit/lmfit-py/pull/287#issuecomment-166794721","issue_url":"https://api.github.com/repos/lmfit/lmfit-py/issues/287","id":166794721,"user":{"login":"andyfaff","id":702934,"avatar_url":"https://avatars.githubusercontent.com/u/702934?v=3","gravatar_id":"","url":"https://api.github.com/users/andyfaff","html_url":"https://github.com/andyfaff","followers_url":"https://api.github.com/users/andyfaff/followers","following_url":"https://api.github.com/users/andyfaff/following{/other_user}","gists_url":"https://api.github.com/users/andyfaff/gists{/gist_id}","starred_url":"https://api.github.com/users/andyfaff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andyfaff/subscriptions","organizations_url":"https://api.github.com/users/andyfaff/orgs","repos_url":"https://api.github.com/users/andyfaff/repos","events_url":"https://api.github.com/users/andyfaff/events{/privacy}","received_events_url":"https://api.github.com/users/andyfaff/received_events","type":"User","site_admin":false},"created_at":"2015-12-23T03:14:46Z","updated_at":"2015-12-23T03:14:46Z","body":"I've since confirmed that a combination of #286 and #287 speedup `Minimizer.__residual` by 37%, for the example script given above. \r\n\r\n(master branch was 0.326 s, both PR's were 0.206 s)"}}