{"action":"opened","issue":{"url":"https://api.github.com/repos/herotc/hero-rotation/issues/265","repository_url":"https://api.github.com/repos/herotc/hero-rotation","labels_url":"https://api.github.com/repos/herotc/hero-rotation/issues/265/labels{/name}","comments_url":"https://api.github.com/repos/herotc/hero-rotation/issues/265/comments","events_url":"https://api.github.com/repos/herotc/hero-rotation/issues/265/events","html_url":"https://github.com/herotc/hero-rotation/issues/265","id":358285599,"node_id":"MDU6SXNzdWUzNTgyODU1OTk=","number":265,"title":"Frost mage should not shatter Ebonbolt. Unless (specific condition)","user":{"login":"towicode","id":8232247,"node_id":"MDQ6VXNlcjgyMzIyNDc=","avatar_url":"https://avatars0.githubusercontent.com/u/8232247?v=4","gravatar_id":"","url":"https://api.github.com/users/towicode","html_url":"https://github.com/towicode","followers_url":"https://api.github.com/users/towicode/followers","following_url":"https://api.github.com/users/towicode/following{/other_user}","gists_url":"https://api.github.com/users/towicode/gists{/gist_id}","starred_url":"https://api.github.com/users/towicode/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/towicode/subscriptions","organizations_url":"https://api.github.com/users/towicode/orgs","repos_url":"https://api.github.com/users/towicode/repos","events_url":"https://api.github.com/users/towicode/events{/privacy}","received_events_url":"https://api.github.com/users/towicode/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-09-08T10:58:32Z","updated_at":"2018-09-08T10:58:32Z","closed_at":null,"author_association":"NONE","body":"The current addon is incorrectly recommending shattering Ebonbolts.\r\n\r\n\r\nHere is what should happen:\r\n\r\n\r\nEbonBolt should be used to generate BrainFreze Proc if we are at 5 Icicles and we don't have BrainFreze.\r\nHowever... this EbonBolt should be shattered. As that BrainFreze should be used for GS.\r\n\r\n\r\nit should go... **5 Icicles > Ebon bolt > GS > Flurry > Ice Lance.**\r\n\r\n\r\nThis is what the addon suggests now.\r\n\r\n\r\n5 Icicles > **Ebon bolt > Flurry > Ice Lance > (wait for another brain freeze) > GS > Flurry > Ice Lance**\r\n\r\n\r\nNow I know what you might be thinking... but it clearly says\r\n```flurry,if=talent.glacial_spike.enabled&buff.brain_freeze.react&(prev_gcd.1.frostbolt&buff.icicles.stack<4|prev_gcd.1.glacial_spike|prev_gcd.1.ebonbolt)\r\n```\r\n\r\nAnd this is the part I don't really understand... (probably because i'm new to simcraft language)\r\n\r\n\r\n**However**,  you look at the (Sample Sequence Table) you'll see the above is true, even inside of SimCraft.\r\n\r\nhttps://i.imgur.com/f0tGl8i.png\r\n\r\n\r\nhttps://i.imgur.com/McGQMOX.png\r\n\r\n\r\nhttps://i.imgur.com/zJfEMOo.png\r\n\r\n\r\n__________________________________________________________________________________\r\n\r\n**There is only 1 case** where you should shatter EbonBolt.\r\n\r\n\r\nThat is if you get a BrainFreze proc on the 4th Icicle. In this case you should do\r\n\r\n\r\n**Frost Bolt x 4 > Ebon Bolt > Flurry > Ice Lance > GS > Flurry > Icelance**\r\n\r\n\r\nYou can see that this also happens in the SIMC sample table.\r\n\r\nhttps://i.imgur.com/fR9YIDX.png"}}
{"actor":{"display_login":"towicode"}}