{"action":"created","issue":{"url":"https://api.github.com/repos/demilich1/metastone/issues/28","labels_url":"https://api.github.com/repos/demilich1/metastone/issues/28/labels{/name}","comments_url":"https://api.github.com/repos/demilich1/metastone/issues/28/comments","events_url":"https://api.github.com/repos/demilich1/metastone/issues/28/events","html_url":"https://github.com/demilich1/metastone/pull/28","id":117409715,"number":28,"title":"Card Updates","user":{"login":"webadict","id":5970174,"avatar_url":"https://avatars.githubusercontent.com/u/5970174?v=3","gravatar_id":"","url":"https://api.github.com/users/webadict","html_url":"https://github.com/webadict","followers_url":"https://api.github.com/users/webadict/followers","following_url":"https://api.github.com/users/webadict/following{/other_user}","gists_url":"https://api.github.com/users/webadict/gists{/gist_id}","starred_url":"https://api.github.com/users/webadict/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/webadict/subscriptions","organizations_url":"https://api.github.com/users/webadict/orgs","repos_url":"https://api.github.com/users/webadict/repos","events_url":"https://api.github.com/users/webadict/events{/privacy}","received_events_url":"https://api.github.com/users/webadict/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-11-17T17:43:48Z","updated_at":"2015-11-23T08:44:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/demilich1/metastone/pulls/28","html_url":"https://github.com/demilich1/metastone/pull/28","diff_url":"https://github.com/demilich1/metastone/pull/28.diff","patch_url":"https://github.com/demilich1/metastone/pull/28.patch"},"body":"Fixes:\r\n-Added remaining LoE cards.\r\n-Fixed Tracking\r\n-Added correct Discover mechanic\r\n\r\nCurrent Identified Issues:\r\n-Rumbling Elemental does not apply AFTER the minion is summoned.\r\n-Reno Jackson does not use a HashSet just yet. (Not really an issue, but I figure it's probably a little bit faster.)\r\n-Mistress of Pain does not correctly heal when ARMOR_SUIT and TAKE_DOUBLE_DAMAGE are active.\r\n\r\nUnknown Value Conditions:\r\n-Are Murlocs resurrected from your opponent with Anyfin Can Happen?"},"comment":{"url":"https://api.github.com/repos/demilich1/metastone/issues/comments/158877869","html_url":"https://github.com/demilich1/metastone/pull/28#issuecomment-158877869","issue_url":"https://api.github.com/repos/demilich1/metastone/issues/28","id":158877869,"user":{"login":"demilich1","id":13793386,"avatar_url":"https://avatars.githubusercontent.com/u/13793386?v=3","gravatar_id":"","url":"https://api.github.com/users/demilich1","html_url":"https://github.com/demilich1","followers_url":"https://api.github.com/users/demilich1/followers","following_url":"https://api.github.com/users/demilich1/following{/other_user}","gists_url":"https://api.github.com/users/demilich1/gists{/gist_id}","starred_url":"https://api.github.com/users/demilich1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/demilich1/subscriptions","organizations_url":"https://api.github.com/users/demilich1/orgs","repos_url":"https://api.github.com/users/demilich1/repos","events_url":"https://api.github.com/users/demilich1/events{/privacy}","received_events_url":"https://api.github.com/users/demilich1/received_events","type":"User","site_admin":false},"created_at":"2015-11-23T08:44:27Z","updated_at":"2015-11-23T08:44:27Z","body":"Thanks a lot implementing all those League of Explorers cards. Much appreciated.\r\n\r\nI just checked out your version and there are still some exceptions being thrown. The first ones I noticed are originating from CopyDeathrattlesSpell and PlayRandomBehaviour (I suppose there are some troubles with the new Discover mechanic), but there are more. I would prefer to delay the merge of the PR until those are fixed. If you want, I can assist you in fixing the exceptions, but that would require that you always push your local commits to github, so we don't do a lot of conflicting local work.\r\n\r\nWhen I am done with some major additions/changes to the codebase, I always run SimulationMode -> Select Behaviour 'PlayRandom' -> set games to 100000 and run. I do this for each class, Warrior vs Warrior 100000 games, Priest vs Priest 100000 games, etc. The idea is to let the AI play every card in the game in many many different situations to find possible bugs. Just a little tip to help you testing. Usually, when all classes complete all 100000 games, the code is pretty stable."}}