It is currently 11 Nov 2025, 10:19
   
Text Size

Bug Reports (snapshot builds)

Post MTG Forge Related Programming Questions Here

Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins

Re: Bug Reports (snapshot builds)

Postby 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 !!!
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)

Postby 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
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby 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
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby 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)

Postby 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.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby 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.
stormcat
 
Posts: 361
Joined: 17 Jun 2015, 05:32
Has thanked: 0 time
Been thanked: 6 times

Re: Bug Reports (snapshot builds)

Postby 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?
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby 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
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby 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"
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby stormcat » 29 Jul 2016, 06:55

@Hanmac
Not doing anything special.
please refer to the attachment.
Attachments
test.jpg
stormcat
 
Posts: 361
Joined: 17 Jun 2015, 05:32
Has thanked: 0 time
Been thanked: 6 times

Re: Bug Reports (snapshot builds)

Postby 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
Agetian
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby 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)
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Agetian » 29 Jul 2016, 10:29

Hanmac 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)
Unfortunately, the AI still somehow finds a way around it, it seems... and still recasts the commander for its original mana cost no matter what :(

- Agetian
Agetian
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby 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.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Agetian » 29 Jul 2016, 11:24

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.
Ah, nice! Thanks! I'll give it a test soon!

- Agetian
Agetian
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

PreviousNext

Return to Developer's Corner

Who is online

Users browsing this forum: Google [Bot] and 9 guests

Main Menu

User Menu

Our Partners


Who is online

In total there are 10 users online :: 1 registered, 0 hidden and 9 guests (based on users active over the past 10 minutes)
Most users ever online was 9824 on 10 Nov 2025, 04:33

Users browsing this forum: Google [Bot] and 9 guests

Login Form