{"action":"created","issue":{"url":"https://api.github.com/repos/desihub/desisim/issues/161","repository_url":"https://api.github.com/repos/desihub/desisim","labels_url":"https://api.github.com/repos/desihub/desisim/issues/161/labels{/name}","comments_url":"https://api.github.com/repos/desihub/desisim/issues/161/comments","events_url":"https://api.github.com/repos/desihub/desisim/issues/161/events","html_url":"https://github.com/desihub/desisim/issues/161","id":177673578,"number":161,"title":"Update quickbrick and quickgen to specsim v0.5","user":{"login":"dkirkby","id":185007,"avatar_url":"https://avatars.githubusercontent.com/u/185007?v=3","gravatar_id":"","url":"https://api.github.com/users/dkirkby","html_url":"https://github.com/dkirkby","followers_url":"https://api.github.com/users/dkirkby/followers","following_url":"https://api.github.com/users/dkirkby/following{/other_user}","gists_url":"https://api.github.com/users/dkirkby/gists{/gist_id}","starred_url":"https://api.github.com/users/dkirkby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dkirkby/subscriptions","organizations_url":"https://api.github.com/users/dkirkby/orgs","repos_url":"https://api.github.com/users/dkirkby/repos","events_url":"https://api.github.com/users/dkirkby/events{/privacy}","received_events_url":"https://api.github.com/users/dkirkby/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":17,"created_at":"2016-09-18T20:07:37Z","updated_at":"2016-10-11T20:22:45Z","closed_at":null,"body":"The specsim exposure time parameter moved from `qsim.instrument` to the new`qsim.observation` in v0.5, but quickbrick and quickgen are still setting the value in `qsim.instrument` (which does not raise any exception, since python allows you to add attributes to an object on the fly).  The end result is that the exposure time is always fixed at 1000s.\r\n\r\nThis issue is to fix this problem, test the fix, and check for any other gotchas with the transition to specsim v0.5.\r\n\r\nThanks to @moustakas for flagging this!"},"comment":{"url":"https://api.github.com/repos/desihub/desisim/issues/comments/253034238","html_url":"https://github.com/desihub/desisim/issues/161#issuecomment-253034238","issue_url":"https://api.github.com/repos/desihub/desisim/issues/161","id":253034238,"user":{"login":"sbailey","id":218471,"avatar_url":"https://avatars.githubusercontent.com/u/218471?v=3","gravatar_id":"","url":"https://api.github.com/users/sbailey","html_url":"https://github.com/sbailey","followers_url":"https://api.github.com/users/sbailey/followers","following_url":"https://api.github.com/users/sbailey/following{/other_user}","gists_url":"https://api.github.com/users/sbailey/gists{/gist_id}","starred_url":"https://api.github.com/users/sbailey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbailey/subscriptions","organizations_url":"https://api.github.com/users/sbailey/orgs","repos_url":"https://api.github.com/users/sbailey/repos","events_url":"https://api.github.com/users/sbailey/events{/privacy}","received_events_url":"https://api.github.com/users/sbailey/received_events","type":"User","site_admin":false},"created_at":"2016-10-11T20:22:45Z","updated_at":"2016-10-11T20:22:45Z","body":"IIUC, that parameter controls the scaling of the *extinction* of the sky lines, but there is also an airmass dependence of the airglow itself.  i.e. higher airmass = more airglow.  e.g. see http://adsabs.harvard.edu/abs/1998NewAR..42..503B"}}
{"actor":{"display_login":"sbailey"}}