{"action":"created","issue":{"url":"https://api.github.com/repos/nanograv/PINT/issues/188","repository_url":"https://api.github.com/repos/nanograv/PINT","labels_url":"https://api.github.com/repos/nanograv/PINT/issues/188/labels{/name}","comments_url":"https://api.github.com/repos/nanograv/PINT/issues/188/comments","events_url":"https://api.github.com/repos/nanograv/PINT/issues/188/events","html_url":"https://github.com/nanograv/PINT/pull/188","id":183460093,"number":188,"title":"Fix wls fitter","user":{"login":"luojing1211","id":5324931,"avatar_url":"https://avatars.githubusercontent.com/u/5324931?v=3","gravatar_id":"","url":"https://api.github.com/users/luojing1211","html_url":"https://github.com/luojing1211","followers_url":"https://api.github.com/users/luojing1211/followers","following_url":"https://api.github.com/users/luojing1211/following{/other_user}","gists_url":"https://api.github.com/users/luojing1211/gists{/gist_id}","starred_url":"https://api.github.com/users/luojing1211/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/luojing1211/subscriptions","organizations_url":"https://api.github.com/users/luojing1211/orgs","repos_url":"https://api.github.com/users/luojing1211/repos","events_url":"https://api.github.com/users/luojing1211/events{/privacy}","received_events_url":"https://api.github.com/users/luojing1211/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":14,"created_at":"2016-10-17T16:12:10Z","updated_at":"2016-11-22T00:00:03Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nanograv/PINT/pulls/188","html_url":"https://github.com/nanograv/PINT/pull/188","diff_url":"https://github.com/nanograv/PINT/pull/188.diff","patch_url":"https://github.com/nanograv/PINT/pull/188.patch"},"body":"Dear all, \nThis is a branch for fixing wls fitter. It is still in the test phase. But I would like our developers get a chance to take a look and make comments. \n\n-Luo Jing \n"},"comment":{"url":"https://api.github.com/repos/nanograv/PINT/issues/comments/262106521","html_url":"https://github.com/nanograv/PINT/pull/188#issuecomment-262106521","issue_url":"https://api.github.com/repos/nanograv/PINT/issues/188","id":262106521,"user":{"login":"demorest","id":159985,"avatar_url":"https://avatars.githubusercontent.com/u/159985?v=3","gravatar_id":"","url":"https://api.github.com/users/demorest","html_url":"https://github.com/demorest","followers_url":"https://api.github.com/users/demorest/followers","following_url":"https://api.github.com/users/demorest/following{/other_user}","gists_url":"https://api.github.com/users/demorest/gists{/gist_id}","starred_url":"https://api.github.com/users/demorest/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/demorest/subscriptions","organizations_url":"https://api.github.com/users/demorest/orgs","repos_url":"https://api.github.com/users/demorest/repos","events_url":"https://api.github.com/users/demorest/events{/privacy}","received_events_url":"https://api.github.com/users/demorest/received_events","type":"User","site_admin":false},"created_at":"2016-11-22T00:00:03Z","updated_at":"2016-11-22T00:00:03Z","body":"@luojing1211 please see the new_fitter branch I just pushed to my repository.  I made changes to the matrix calculations in wls_fitter.call_minimize() to make it more numerically stable, as we discussed on the phone.  The main changes were to mean-subtract and re-scale the design matrix columns before doing anything else; and compute SVD of the design matrix itself rather than of the \"M^T M\" product.\r\n\r\nThis version returns parameter uncertainties that are mostly within ~10% of the tempo2 values using the B1855 5-year test data set.  Except for RAJ and DECJ, where I think there is a units problem (probably arcseconds vs degrees or similar)."}}
{"actor":{"display_login":"demorest"}}