{"action":"created","issue":{"url":"https://api.github.com/repos/spencerahill/aospy/issues/74","repository_url":"https://api.github.com/repos/spencerahill/aospy","labels_url":"https://api.github.com/repos/spencerahill/aospy/issues/74/labels{/name}","comments_url":"https://api.github.com/repos/spencerahill/aospy/issues/74/comments","events_url":"https://api.github.com/repos/spencerahill/aospy/issues/74/events","html_url":"https://github.com/spencerahill/aospy/pull/74","id":156074665,"number":74,"title":"Misc. updates I've made over the past two months","user":{"login":"spencerahill","id":6200806,"avatar_url":"https://avatars.githubusercontent.com/u/6200806?v=3","gravatar_id":"","url":"https://api.github.com/users/spencerahill","html_url":"https://github.com/spencerahill","followers_url":"https://api.github.com/users/spencerahill/followers","following_url":"https://api.github.com/users/spencerahill/following{/other_user}","gists_url":"https://api.github.com/users/spencerahill/gists{/gist_id}","starred_url":"https://api.github.com/users/spencerahill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spencerahill/subscriptions","organizations_url":"https://api.github.com/users/spencerahill/orgs","repos_url":"https://api.github.com/users/spencerahill/repos","events_url":"https://api.github.com/users/spencerahill/events{/privacy}","received_events_url":"https://api.github.com/users/spencerahill/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-05-21T00:46:17Z","updated_at":"2016-05-21T01:08:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/spencerahill/aospy/pulls/74","html_url":"https://github.com/spencerahill/aospy/pull/74","diff_url":"https://github.com/spencerahill/aospy/pull/74.diff","patch_url":"https://github.com/spencerahill/aospy/pull/74.patch"},"body":"This is very poor form on my part to dump all these at once...got caught up in my thesis.  Taking a mini breather this afternoon and thought I'd push these.  To make matters worse, I don't have any tests.\r\n\r\n@spencerkclark Would you mind checking out this branch (in both the informal and git senses)?  I'll update as needed to jive with #73 once it's merged (if the tests ever finish running ;)).  But just let me know if you see anything glaring.  Please wait until AFTER your generals exam...absolutely no rush on this\r\n\r\nRe: backtracking on my previous pledge to not merge things in without tests: realistically it is not feasible for me to take the needed time to write tests until after August 2.  So I'll continue (with apologies) to be sloppy until then.  After the defense, in addition to the code revamp, I'd like to revamp the whole approach to this package, treating it like the legit open source, multi-user package it is, rather than my personal research tool that happens to be on github.\r\n\r\nSome of the more relevant changes:\r\n* Use `np.allclose` to determine if coordinates (namely `lat`) differ by very little between a Model and the given data; if they do then just overwrite the coord in the data with the Model's value\r\n* Several bugfixes and added functionality in `plotting`\r\n* Improve/make more consistent handling of logging/warnings\r\n* Save region-specific pressure data for data on model coordinates\r\n* Compute dt from the time bounds, rather than just assuming equal time spacing if the data lacks a dt array\r\n* Bugfixes in `utils` `level_thickness` and `dp_from_ps` functions\r\n* Bugfixes in surface area\r\n "},"comment":{"url":"https://api.github.com/repos/spencerahill/aospy/issues/comments/220750052","html_url":"https://github.com/spencerahill/aospy/pull/74#issuecomment-220750052","issue_url":"https://api.github.com/repos/spencerahill/aospy/issues/74","id":220750052,"user":{"login":"spencerkclark","id":6628425,"avatar_url":"https://avatars.githubusercontent.com/u/6628425?v=3","gravatar_id":"","url":"https://api.github.com/users/spencerkclark","html_url":"https://github.com/spencerkclark","followers_url":"https://api.github.com/users/spencerkclark/followers","following_url":"https://api.github.com/users/spencerkclark/following{/other_user}","gists_url":"https://api.github.com/users/spencerkclark/gists{/gist_id}","starred_url":"https://api.github.com/users/spencerkclark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spencerkclark/subscriptions","organizations_url":"https://api.github.com/users/spencerkclark/orgs","repos_url":"https://api.github.com/users/spencerkclark/repos","events_url":"https://api.github.com/users/spencerkclark/events{/privacy}","received_events_url":"https://api.github.com/users/spencerkclark/received_events","type":"User","site_admin":false},"created_at":"2016-05-21T01:08:09Z","updated_at":"2016-05-21T01:08:09Z","body":"No worries -- I'll have a closer look a these changes sometime in the next few weeks."}}
{"actor":{"display_login":"spencerkclark"}}