Bug Reports (snapshot builds)
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins
Re: Bug Reports (snapshot builds)
by fmartel » 28 Jul 2016, 15:34
in r31736 each time I click to recast my commander (from the Zone, not being killed) It cost me 2C more each time. Now, my commander is costing 14C+normal cost
in -r31755 it adds 4C more !!!
in -r31755 it adds 4C more !!!
Last edited by fmartel on 28 Jul 2016, 16:31, edited 1 time in total.
- fmartel
- Posts: 281
- Joined: 31 Dec 2013, 19:27
- Location: Québec City
- Has thanked: 8 times
- Been thanked: 4 times
Re: Bug Reports (snapshot builds)
by Agetian » 28 Jul 2016, 16:25
r31755: Evoke appears not to work. I cast Shriekmaw with Evoke and it did not get sacrificed as it entered the battlefield.
- Agetian
- Agetian
- Agetian
- Programmer
- Posts: 3490
- Joined: 14 Mar 2011, 05:58
- Has thanked: 684 times
- Been thanked: 572 times
Re: Bug Reports (snapshot builds)
by Hanmac » 28 Jul 2016, 17:04
like i thought, Evoke was totally my fault ... i tested it with the only creature i forget to convert :/
now back again to my MayPlay project ... i need to finish it before next release
now back again to my MayPlay project ... i need to finish it before next release
Re: Bug Reports (snapshot builds)
by fmartel » 28 Jul 2016, 18:09
-r31757 I still have the problem where If I try to re-cast my commander the cost goes up each time I check it. (died once, but try to cast it a second time, not enough mana, check again 2C mana has been added)
- fmartel
- Posts: 281
- Joined: 31 Dec 2013, 19:27
- Location: Québec City
- Has thanked: 8 times
- Been thanked: 4 times
Re: Bug Reports (snapshot builds)
by Hanmac » 28 Jul 2016, 18:31
@fmartel: its only normal that you still have the problem when between
r31736 and r31757 it hasn't been fixed yet.
there is the svn changelog:
SVN changelog
i will take a look at it: from my current watching the parameters with Amount does look like they should.
hm while i am there, i might rewrite it a bit about the "May be played" stuff
Edit: Commander RaiseCost should be fixed now, i also updated the MayPlay stuff.
r31736 and r31757 it hasn't been fixed yet.
there is the svn changelog:
SVN changelog
i will take a look at it: from my current watching the parameters with Amount does look like they should.
hm while i am there, i might rewrite it a bit about the "May be played" stuff
Edit: Commander RaiseCost should be fixed now, i also updated the MayPlay stuff.
Re: Bug Reports (snapshot builds)
by stormcat » 28 Jul 2016, 21:01
:r31760
In Commander Game
I controlled Mikaeus, the Unhallowed and Triskelion.
When Triskelion with a undying dies, triggered double.
In Commander Game
I controlled Mikaeus, the Unhallowed and Triskelion.
When Triskelion with a undying dies, triggered double.
Re: Bug Reports (snapshot builds)
by Hanmac » 29 Jul 2016, 05:43
@stormcat: i tried to reproduce it, but i didn't get your result yet.
can you try it again and maybe dump the game state or give a step-by-step how you get that with two triggers?
can you try it again and maybe dump the game state or give a step-by-step how you get that with two triggers?
Re: Bug Reports (snapshot builds)
by Agetian » 29 Jul 2016, 05:52
r31760: For some reason, when trying to cast a commander from Command Zone (any commander, tried with three different ones), I am presented with a list of two options to cast the commander, both of them the same. Clicking either one works. This is minor but I think this should not happen...
- Agetian
- Agetian
- Agetian
- Programmer
- Posts: 3490
- Joined: 14 Mar 2011, 05:58
- Has thanked: 684 times
- Been thanked: 572 times
Re: Bug Reports (snapshot builds)
by Hanmac » 29 Jul 2016, 06:07
Agentian: that is a problem with the MayPlay stuff,
for some reason the original SA does gets a true for checkZoneRestrictions
see this post in another thread:
viewtopic.php?f=52&t=3760&p=199568#p199567
i didn't had the time yet to check this out,
i will probably when i converted more stuff from "May be played" into "MayPlay"
for some reason the original SA does gets a true for checkZoneRestrictions
see this post in another thread:
viewtopic.php?f=52&t=3760&p=199568#p199567
i didn't had the time yet to check this out,
i will probably when i converted more stuff from "May be played" into "MayPlay"
Re: Bug Reports (snapshot builds)
by stormcat » 29 Jul 2016, 06:55
@Hanmac
Not doing anything special.
please refer to the attachment.
Not doing anything special.
please refer to the attachment.
Re: Bug Reports (snapshot builds)
by Agetian » 29 Jul 2016, 09:50
I think I may have found a bigger problem with recasting the Commander from the command zone, at least kind of. I'm playing a lot of Commander and Planeswalker mode on mobile Forge in Planar Conquest mode, so this has become rather obvious with updating to the latest snapshot build (SVN r31760, in particular):
The AI can recast the commander and the planeswalker without the raised cost, e.g. it always casts Kiora, the Crashing Wave in planeswalker mode for 4 mana, which gives the AI quite a bit of unfair advantage especially as the game goes on. The same happens with commanders (tested with quite a few), the AI can always recast the commander for its original mana cost. This used not to happen recently... not sure if it's a part of the same MayPlay issue as mentioned before or a separate thing.
- Agetian
The AI can recast the commander and the planeswalker without the raised cost, e.g. it always casts Kiora, the Crashing Wave in planeswalker mode for 4 mana, which gives the AI quite a bit of unfair advantage especially as the game goes on. The same happens with commanders (tested with quite a few), the AI can always recast the commander for its original mana cost. This used not to happen recently... not sure if it's a part of the same MayPlay issue as mentioned before or a separate thing.
- Agetian
- Agetian
- Programmer
- Posts: 3490
- Joined: 14 Mar 2011, 05:58
- Has thanked: 684 times
- Been thanked: 572 times
Re: Bug Reports (snapshot builds)
by Hanmac » 29 Jul 2016, 10:22
Agetian: i added a fix in checkZoneRestrictions, so mayPlay only for nonBasicSpells.
now the BasicSA should be blocked for the mayPlay stuff (until i rewrite it more)
i am not 100% sure if that does block it for the AI.
(or the AI does find a way to cheat)
now the BasicSA should be blocked for the mayPlay stuff (until i rewrite it more)
i am not 100% sure if that does block it for the AI.
(or the AI does find a way to cheat)
Re: Bug Reports (snapshot builds)
by Agetian » 29 Jul 2016, 10:29
Unfortunately, the AI still somehow finds a way around it, it seems... and still recasts the commander for its original mana cost no matter whatHanmac wrote:Agetian: i added a fix in checkZoneRestrictions, so mayPlay only for nonBasicSpells.
now the BasicSA should be blocked for the mayPlay stuff (until i rewrite it more)
i am not 100% sure if that does block it for the AI.
(or the AI does find a way to cheat)
- Agetian
- Agetian
- Programmer
- Posts: 3490
- Joined: 14 Mar 2011, 05:58
- Has thanked: 684 times
- Been thanked: 572 times
Re: Bug Reports (snapshot builds)
by Hanmac » 29 Jul 2016, 11:19
Agetian: i think i found it, there was an adjust missing in:
ComputerUtilMana.calculateManaCost
so the AI got away without raising ManaCost, now it should be fixed.
ComputerUtilMana.calculateManaCost
so the AI got away without raising ManaCost, now it should be fixed.
Re: Bug Reports (snapshot builds)
by Agetian » 29 Jul 2016, 11:24
Ah, nice! Thanks! I'll give it a test soon!Hanmac wrote:Agetian: i think i found it, there was an adjust missing in:
ComputerUtilMana.calculateManaCost
so the AI got away without raising ManaCost, now it should be fixed.
- Agetian
- Agetian
- Programmer
- Posts: 3490
- Joined: 14 Mar 2011, 05:58
- Has thanked: 684 times
- Been thanked: 572 times
Who is online
Users browsing this forum: Google [Bot] and 9 guests