{"action":"created","issue":{"url":"https://api.github.com/repos/desihub/redrock/issues/106","repository_url":"https://api.github.com/repos/desihub/redrock","labels_url":"https://api.github.com/repos/desihub/redrock/issues/106/labels{/name}","comments_url":"https://api.github.com/repos/desihub/redrock/issues/106/comments","events_url":"https://api.github.com/repos/desihub/redrock/issues/106/events","html_url":"https://github.com/desihub/redrock/issues/106","id":309900804,"node_id":"MDU6SXNzdWUzMDk5MDA4MDQ=","number":106,"title":"new QSO templates without LyA","user":{"login":"sbailey","id":218471,"node_id":"MDQ6VXNlcjIxODQ3MQ==","avatar_url":"https://avatars3.githubusercontent.com/u/218471?v=4","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},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-03-29T20:55:44Z","updated_at":"2018-12-07T10:56:50Z","closed_at":null,"author_association":"COLLABORATOR","body":"PR #104 corrects for mean LyA while doing the QSO redshift scan, but it still uses the old QSO templates from BOSS that didn't remove LyA while building them.  Create a new set of QSO templates that also remove mean LyA while building the templates.\r\n\r\nPR #104 showed an example of this, but there was a strange notch in the templates at LyA, appearing as if some of the input QSOs had the wrong z and were thus mis-corrected.\r\n\r\nThese updated templates along with #104 should provide more consistent results on real data, but should also be coordinated with an update to simqso (or the desispec wrapper of simqso) since those templates also include mean LyA, and then additional LyA absorption is included on top of that (ugh)."},"comment":{"url":"https://api.github.com/repos/desihub/redrock/issues/comments/445197325","html_url":"https://github.com/desihub/redrock/issues/106#issuecomment-445197325","issue_url":"https://api.github.com/repos/desihub/redrock/issues/106","id":445197325,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0NTE5NzMyNQ==","user":{"login":"moustakas","id":1431820,"node_id":"MDQ6VXNlcjE0MzE4MjA=","avatar_url":"https://avatars2.githubusercontent.com/u/1431820?v=4","gravatar_id":"","url":"https://api.github.com/users/moustakas","html_url":"https://github.com/moustakas","followers_url":"https://api.github.com/users/moustakas/followers","following_url":"https://api.github.com/users/moustakas/following{/other_user}","gists_url":"https://api.github.com/users/moustakas/gists{/gist_id}","starred_url":"https://api.github.com/users/moustakas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moustakas/subscriptions","organizations_url":"https://api.github.com/users/moustakas/orgs","repos_url":"https://api.github.com/users/moustakas/repos","events_url":"https://api.github.com/users/moustakas/events{/privacy}","received_events_url":"https://api.github.com/users/moustakas/received_events","type":"User","site_admin":false},"created_at":"2018-12-07T10:56:50Z","updated_at":"2018-12-07T10:56:50Z","author_association":"MEMBER","body":"It would be great to get these templates into `desisim`, not just `redrock`...."}}
{"actor":{"display_login":"moustakas"}}