{"action":"created","issue":{"url":"https://api.github.com/repos/boostorg/graph/issues/114","repository_url":"https://api.github.com/repos/boostorg/graph","labels_url":"https://api.github.com/repos/boostorg/graph/issues/114/labels{/name}","comments_url":"https://api.github.com/repos/boostorg/graph/issues/114/comments","events_url":"https://api.github.com/repos/boostorg/graph/issues/114/events","html_url":"https://github.com/boostorg/graph/pull/114","id":356169021,"node_id":"MDExOlB1bGxSZXF1ZXN0MjEyNTI3MjA3","number":114,"title":"First batch of merges from the backlog.","user":{"login":"anadon","id":3113477,"node_id":"MDQ6VXNlcjMxMTM0Nzc=","avatar_url":"https://avatars3.githubusercontent.com/u/3113477?v=4","gravatar_id":"","url":"https://api.github.com/users/anadon","html_url":"https://github.com/anadon","followers_url":"https://api.github.com/users/anadon/followers","following_url":"https://api.github.com/users/anadon/following{/other_user}","gists_url":"https://api.github.com/users/anadon/gists{/gist_id}","starred_url":"https://api.github.com/users/anadon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anadon/subscriptions","organizations_url":"https://api.github.com/users/anadon/orgs","repos_url":"https://api.github.com/users/anadon/repos","events_url":"https://api.github.com/users/anadon/events{/privacy}","received_events_url":"https://api.github.com/users/anadon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2018-09-01T04:00:55Z","updated_at":"2018-09-01T17:27:29Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/boostorg/graph/pulls/114","html_url":"https://github.com/boostorg/graph/pull/114","diff_url":"https://github.com/boostorg/graph/pull/114.diff","patch_url":"https://github.com/boostorg/graph/pull/114.patch"},"body":"Added PR's #80, #93, #95, #111  These are well formed bugfix PR's which should be added.  All tests are passing locally, but I don't have the facilities to test more broadly.  The broader test facilities would be a future feature addition and out of scope of what I've been immediately tasked with."},"comment":{"url":"https://api.github.com/repos/boostorg/graph/issues/comments/417874824","html_url":"https://github.com/boostorg/graph/pull/114#issuecomment-417874824","issue_url":"https://api.github.com/repos/boostorg/graph/issues/114","id":417874824,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNzg3NDgyNA==","user":{"login":"jzmaddock","id":5011768,"node_id":"MDQ6VXNlcjUwMTE3Njg=","avatar_url":"https://avatars2.githubusercontent.com/u/5011768?v=4","gravatar_id":"","url":"https://api.github.com/users/jzmaddock","html_url":"https://github.com/jzmaddock","followers_url":"https://api.github.com/users/jzmaddock/followers","following_url":"https://api.github.com/users/jzmaddock/following{/other_user}","gists_url":"https://api.github.com/users/jzmaddock/gists{/gist_id}","starred_url":"https://api.github.com/users/jzmaddock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jzmaddock/subscriptions","organizations_url":"https://api.github.com/users/jzmaddock/orgs","repos_url":"https://api.github.com/users/jzmaddock/repos","events_url":"https://api.github.com/users/jzmaddock/events{/privacy}","received_events_url":"https://api.github.com/users/jzmaddock/received_events","type":"User","site_admin":false},"created_at":"2018-09-01T17:27:29Z","updated_at":"2018-09-01T17:27:29Z","author_association":"CONTRIBUTOR","body":">It actually looks like the relaxed_heap is not used elsewhere, users don't have a method to enable it, a google search yields no used examples for a total of 6 results, and related structures in code are just not used. With it being vestigial, I'd like to remove it. I'd like a go-ahead before I muck around with that.\r\n\r\nA quick bit of grepping around: There are a few headers which include it, but don't use it, those #includes should be removed.\r\n\r\nIt is still used in dijkstra_shortest_Paths.hpp if BOOST_GRAPH_DIJKSTRA_USE_RELAXED_HEAP is defined, but this is not the default, and it appears not to be documented anywhere.\r\n\r\nSo... my suggestion would be to remove all references to it, but leave the header in place, and add some #pragmas to it to loudly report it as deprecated and unsupported."}}
{"actor":{"display_login":"jzmaddock"}}