{"action":"created","issue":{"url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/266","repository_url":"https://api.github.com/repos/EmmittJ/PoESkillTree","labels_url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/266/labels{/name}","comments_url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/266/comments","events_url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/266/events","html_url":"https://github.com/EmmittJ/PoESkillTree/pull/266","id":137621605,"number":266,"title":"Fix for #251","user":{"login":"MLanghof","id":6524684,"avatar_url":"https://avatars.githubusercontent.com/u/6524684?v=3","gravatar_id":"","url":"https://api.github.com/users/MLanghof","html_url":"https://github.com/MLanghof","followers_url":"https://api.github.com/users/MLanghof/followers","following_url":"https://api.github.com/users/MLanghof/following{/other_user}","gists_url":"https://api.github.com/users/MLanghof/gists{/gist_id}","starred_url":"https://api.github.com/users/MLanghof/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MLanghof/subscriptions","organizations_url":"https://api.github.com/users/MLanghof/orgs","repos_url":"https://api.github.com/users/MLanghof/repos","events_url":"https://api.github.com/users/MLanghof/events{/privacy}","received_events_url":"https://api.github.com/users/MLanghof/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-03-01T16:29:08Z","updated_at":"2016-03-01T18:53:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/EmmittJ/PoESkillTree/pulls/266","html_url":"https://github.com/EmmittJ/PoESkillTree/pull/266","diff_url":"https://github.com/EmmittJ/PoESkillTree/pull/266.diff","patch_url":"https://github.com/EmmittJ/PoESkillTree/pull/266.patch"},"body":"Fixing #251.\r\n\r\nThe algorithm now ignores all Ascendancy nodes as potential pathing nodes.\r\nThis also makes sense, because players can *really* just skill the Ascendancy nodes themselves.\r\n\r\nIt would make sense to disable the tagging of Ascendancy nodes altogether, but I don't want to make that decision myself."},"comment":{"url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/comments/190850569","html_url":"https://github.com/EmmittJ/PoESkillTree/pull/266#issuecomment-190850569","issue_url":"https://api.github.com/repos/EmmittJ/PoESkillTree/issues/266","id":190850569,"user":{"login":"EmmittJ","id":7478125,"avatar_url":"https://avatars.githubusercontent.com/u/7478125?v=3","gravatar_id":"","url":"https://api.github.com/users/EmmittJ","html_url":"https://github.com/EmmittJ","followers_url":"https://api.github.com/users/EmmittJ/followers","following_url":"https://api.github.com/users/EmmittJ/following{/other_user}","gists_url":"https://api.github.com/users/EmmittJ/gists{/gist_id}","starred_url":"https://api.github.com/users/EmmittJ/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EmmittJ/subscriptions","organizations_url":"https://api.github.com/users/EmmittJ/orgs","repos_url":"https://api.github.com/users/EmmittJ/repos","events_url":"https://api.github.com/users/EmmittJ/events{/privacy}","received_events_url":"https://api.github.com/users/EmmittJ/received_events","type":"User","site_admin":false},"created_at":"2016-03-01T18:53:15Z","updated_at":"2016-03-01T18:53:15Z","body":"I think removing ascendancy tagging is a good idea. If someone really\nwanted the scion ascendancy stuff and start at a new place they can just\nskill 'Path of the ...'. The algorithm should work since the starting nodes\nare a possible 'out'\nOn Mar 1, 2016 10:29 AM, \"Max Langhof\" <notifications@github.com> wrote:\n\n> Well it certainly should be an easy fix. I can just undo the \"error\n> checking\" and prevent the ascendancy tagging, which should deal with the\n> issue.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/EmmittJ/PoESkillTree/pull/266#issuecomment-190843284>.\n>\n"}}