{"action":"created","issue":{"url":"https://api.github.com/repos/wbinventor/openmc/issues/6","labels_url":"https://api.github.com/repos/wbinventor/openmc/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/wbinventor/openmc/issues/6/comments","events_url":"https://api.github.com/repos/wbinventor/openmc/issues/6/events","html_url":"https://github.com/wbinventor/openmc/pull/6","id":60598279,"number":6,"title":"Hex lattice for Python API","user":{"login":"smharper","id":4622797,"avatar_url":"https://avatars.githubusercontent.com/u/4622797?v=3","gravatar_id":"","url":"https://api.github.com/users/smharper","html_url":"https://github.com/smharper","followers_url":"https://api.github.com/users/smharper/followers","following_url":"https://api.github.com/users/smharper/following{/other_user}","gists_url":"https://api.github.com/users/smharper/gists{/gist_id}","starred_url":"https://api.github.com/users/smharper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smharper/subscriptions","organizations_url":"https://api.github.com/users/smharper/orgs","repos_url":"https://api.github.com/users/smharper/repos","events_url":"https://api.github.com/users/smharper/events{/privacy}","received_events_url":"https://api.github.com/users/smharper/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-03-11T01:49:58Z","updated_at":"2015-03-17T01:20:23Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/wbinventor/openmc/pulls/6","html_url":"https://github.com/wbinventor/openmc/pull/6","diff_url":"https://github.com/wbinventor/openmc/pull/6.diff","patch_url":"https://github.com/wbinventor/openmc/pull/6.patch"},"body":"This PR will add hex lattice capability to the python API.  It's not complete on it's own; here are the unresolved issues:\r\n\r\n- [x] Make the example a little more clear; double check that it will run properly\r\n- [x] Fix `__repr__`\r\n- [x] Add length checking for `set_universes`\r\n- [x] Add whitespace padding to `repr_axial_slice` to make it more readable\r\n\r\nBefore I address those issues I want to double-check with you that the way I am indexing the universe list is okay.  It looks like you meant for the list to be indexed is `universes[ring #][angular #][axial #]`, but I think it will be much easier if we use the scheme `universes[axial #][ring #][angular #]`.  That second scheme is the one I used here because it made it easy to use a function like `_repr_axial_slice`, but I'm sure I can change to the first scheme if you strongly prefer it."},"comment":{"url":"https://api.github.com/repos/wbinventor/openmc/issues/comments/82017966","html_url":"https://github.com/wbinventor/openmc/pull/6#issuecomment-82017966","issue_url":"https://api.github.com/repos/wbinventor/openmc/issues/6","id":82017966,"user":{"login":"smharper","id":4622797,"avatar_url":"https://avatars.githubusercontent.com/u/4622797?v=3","gravatar_id":"","url":"https://api.github.com/users/smharper","html_url":"https://github.com/smharper","followers_url":"https://api.github.com/users/smharper/followers","following_url":"https://api.github.com/users/smharper/following{/other_user}","gists_url":"https://api.github.com/users/smharper/gists{/gist_id}","starred_url":"https://api.github.com/users/smharper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smharper/subscriptions","organizations_url":"https://api.github.com/users/smharper/orgs","repos_url":"https://api.github.com/users/smharper/repos","events_url":"https://api.github.com/users/smharper/events{/privacy}","received_events_url":"https://api.github.com/users/smharper/received_events","type":"User","site_admin":false},"created_at":"2015-03-17T01:20:23Z","updated_at":"2015-03-17T01:20:23Z","body":"Thanks for the quick reviews.  I think I have addressed all of your comments."}}