{"action":"created","issue":{"url":"https://api.github.com/repos/GalSim-developers/GalSim/issues/722","repository_url":"https://api.github.com/repos/GalSim-developers/GalSim","labels_url":"https://api.github.com/repos/GalSim-developers/GalSim/issues/722/labels{/name}","comments_url":"https://api.github.com/repos/GalSim-developers/GalSim/issues/722/comments","events_url":"https://api.github.com/repos/GalSim-developers/GalSim/issues/722/events","html_url":"https://github.com/GalSim-developers/GalSim/issues/722","id":140331003,"number":722,"title":"Using results of Poisson CCD Simulator to simulate BF effect","user":{"login":"arunkannawadi","id":5103390,"avatar_url":"https://avatars.githubusercontent.com/u/5103390?v=3","gravatar_id":"","url":"https://api.github.com/users/arunkannawadi","html_url":"https://github.com/arunkannawadi","followers_url":"https://api.github.com/users/arunkannawadi/followers","following_url":"https://api.github.com/users/arunkannawadi/following{/other_user}","gists_url":"https://api.github.com/users/arunkannawadi/gists{/gist_id}","starred_url":"https://api.github.com/users/arunkannawadi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arunkannawadi/subscriptions","organizations_url":"https://api.github.com/users/arunkannawadi/orgs","repos_url":"https://api.github.com/users/arunkannawadi/repos","events_url":"https://api.github.com/users/arunkannawadi/events{/privacy}","received_events_url":"https://api.github.com/users/arunkannawadi/received_events","type":"User","site_admin":false},"labels":[{"id":83487880,"url":"https://api.github.com/repos/GalSim-developers/GalSim/labels/feature%20request","name":"feature request","color":"d4c5f9","default":false},{"id":1403459,"url":"https://api.github.com/repos/GalSim-developers/GalSim/labels/module","name":"module","color":"02e10c","default":false}],"state":"open","locked":false,"assignee":{"login":"craiglagegit","id":9419258,"avatar_url":"https://avatars.githubusercontent.com/u/9419258?v=3","gravatar_id":"","url":"https://api.github.com/users/craiglagegit","html_url":"https://github.com/craiglagegit","followers_url":"https://api.github.com/users/craiglagegit/followers","following_url":"https://api.github.com/users/craiglagegit/following{/other_user}","gists_url":"https://api.github.com/users/craiglagegit/gists{/gist_id}","starred_url":"https://api.github.com/users/craiglagegit/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/craiglagegit/subscriptions","organizations_url":"https://api.github.com/users/craiglagegit/orgs","repos_url":"https://api.github.com/users/craiglagegit/repos","events_url":"https://api.github.com/users/craiglagegit/events{/privacy}","received_events_url":"https://api.github.com/users/craiglagegit/received_events","type":"User","site_admin":false},"assignees":[{"login":"craiglagegit","id":9419258,"avatar_url":"https://avatars.githubusercontent.com/u/9419258?v=3","gravatar_id":"","url":"https://api.github.com/users/craiglagegit","html_url":"https://github.com/craiglagegit","followers_url":"https://api.github.com/users/craiglagegit/followers","following_url":"https://api.github.com/users/craiglagegit/following{/other_user}","gists_url":"https://api.github.com/users/craiglagegit/gists{/gist_id}","starred_url":"https://api.github.com/users/craiglagegit/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/craiglagegit/subscriptions","organizations_url":"https://api.github.com/users/craiglagegit/orgs","repos_url":"https://api.github.com/users/craiglagegit/repos","events_url":"https://api.github.com/users/craiglagegit/events{/privacy}","received_events_url":"https://api.github.com/users/craiglagegit/received_events","type":"User","site_admin":false}],"milestone":null,"comments":48,"created_at":"2016-03-12T01:25:13Z","updated_at":"2016-11-14T19:01:51Z","closed_at":null,"body":"Craig Lage at UC Davis has developed a Poisson equation solver that computes the distortion of the pixel edges as the electrons get collected in the CCD pixels. In the DESC Hack day today, we made some progress towards integrating his code (in C++) with the C++ layer of the GalSim. We identified  that the natural way to include this functionality would be by tweaking the photon-shooting method inside the `drawImage` routine.  I will push the code after some cleanup, but this is not completely done yet.\n"},"comment":{"url":"https://api.github.com/repos/GalSim-developers/GalSim/issues/comments/260428332","html_url":"https://github.com/GalSim-developers/GalSim/issues/722#issuecomment-260428332","issue_url":"https://api.github.com/repos/GalSim-developers/GalSim/issues/722","id":260428332,"user":{"login":"rmjarvis","id":623887,"avatar_url":"https://avatars.githubusercontent.com/u/623887?v=3","gravatar_id":"","url":"https://api.github.com/users/rmjarvis","html_url":"https://github.com/rmjarvis","followers_url":"https://api.github.com/users/rmjarvis/followers","following_url":"https://api.github.com/users/rmjarvis/following{/other_user}","gists_url":"https://api.github.com/users/rmjarvis/gists{/gist_id}","starred_url":"https://api.github.com/users/rmjarvis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rmjarvis/subscriptions","organizations_url":"https://api.github.com/users/rmjarvis/orgs","repos_url":"https://api.github.com/users/rmjarvis/repos","events_url":"https://api.github.com/users/rmjarvis/events{/privacy}","received_events_url":"https://api.github.com/users/rmjarvis/received_events","type":"User","site_admin":false},"created_at":"2016-11-14T19:01:51Z","updated_at":"2016-11-14T19:01:51Z","body":"You could put the memory demands in the doc string of the SiliconSensor class in Python so the user is aware of them.  But no, you don't need to explicitly check the memory.  If it runs out of memory, it will raise an exception and let the user know why it failed.  Then they can decide what they want to do about that.\r\n\r\nIf it becomes onerous to deal with this, we can reevaluate the memory considerations as a separate issue.  For now, I think just focus on getting everything working with a robust set of tests.  That makes it a lot easier to try out various optimizations later and be confident that you haven't broken anything."}}
{"actor":{"display_login":"rmjarvis"}}