{"action":"created","issue":{"url":"https://api.github.com/repos/PMEAL/OpenPNM/issues/459","repository_url":"https://api.github.com/repos/PMEAL/OpenPNM","labels_url":"https://api.github.com/repos/PMEAL/OpenPNM/issues/459/labels{/name}","comments_url":"https://api.github.com/repos/PMEAL/OpenPNM/issues/459/comments","events_url":"https://api.github.com/repos/PMEAL/OpenPNM/issues/459/events","html_url":"https://github.com/PMEAL/OpenPNM/issues/459","id":119745034,"number":459,"title":"Access Limited OP bug","user":{"login":"TomTranter","id":7068741,"avatar_url":"https://avatars.githubusercontent.com/u/7068741?v=3","gravatar_id":"","url":"https://api.github.com/users/TomTranter","html_url":"https://github.com/TomTranter","followers_url":"https://api.github.com/users/TomTranter/followers","following_url":"https://api.github.com/users/TomTranter/following{/other_user}","gists_url":"https://api.github.com/users/TomTranter/gists{/gist_id}","starred_url":"https://api.github.com/users/TomTranter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomTranter/subscriptions","organizations_url":"https://api.github.com/users/TomTranter/orgs","repos_url":"https://api.github.com/users/TomTranter/repos","events_url":"https://api.github.com/users/TomTranter/events{/privacy}","received_events_url":"https://api.github.com/users/TomTranter/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/PMEAL/OpenPNM/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/PMEAL/OpenPNM/labels/development%20discussion","name":"development discussion","color":"009800"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-12-01T15:49:34Z","updated_at":"2016-02-09T11:47:36Z","closed_at":"2016-02-09T11:47:36Z","body":"I think there's a bug where if access limited is set to False non-invaded clusters are included in the invaded_clusters. This would be a one line change to put the following line outside the if statement in _do_one_iteration()\r\ninv_clusters = inv_clusters[inv_clusters >= 0]\r\nWhilst I've been looking at this though it occurred to me that we should develop it to do bond and site percolation as it currently just does bond or we should rename it drainage."},"comment":{"url":"https://api.github.com/repos/PMEAL/OpenPNM/issues/comments/181830885","html_url":"https://github.com/PMEAL/OpenPNM/issues/459#issuecomment-181830885","issue_url":"https://api.github.com/repos/PMEAL/OpenPNM/issues/459","id":181830885,"user":{"login":"TomTranter","id":7068741,"avatar_url":"https://avatars.githubusercontent.com/u/7068741?v=3","gravatar_id":"","url":"https://api.github.com/users/TomTranter","html_url":"https://github.com/TomTranter","followers_url":"https://api.github.com/users/TomTranter/followers","following_url":"https://api.github.com/users/TomTranter/following{/other_user}","gists_url":"https://api.github.com/users/TomTranter/gists{/gist_id}","starred_url":"https://api.github.com/users/TomTranter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomTranter/subscriptions","organizations_url":"https://api.github.com/users/TomTranter/orgs","repos_url":"https://api.github.com/users/TomTranter/repos","events_url":"https://api.github.com/users/TomTranter/events{/privacy}","received_events_url":"https://api.github.com/users/TomTranter/received_events","type":"User","site_admin":false},"created_at":"2016-02-09T11:47:36Z","updated_at":"2016-02-09T11:47:36Z","body":"Fixed as part of #466 "}}