It is currently 24 Apr 2024, 19:59
   
Text Size

Born of the Gods Spoiler Season

Post MTG Forge Related Programming Questions Here

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

Re: Born of the Gods Spoiler Season

Postby gos » 31 Jan 2014, 09:40

Max mtg wrote:The one thing that is unclear: Does "until the next time a player would receive priority" stand for the beginning of such a phase when priority can be given to players, or for the actual moment when a player receives priority?
The actual moment he receives priority, it would seem.

Max mtg wrote:That is: 2 players, each has God-Favored General. On untap triggers are remembered, but not placed into stack. During upkeep player1 (whose turn is now) places his triggers onto stack and then has to pass before the other player is allowed to add his tirggers. It that correct? Or the second player start arranging his triggers without waiting for his opponent one to pass?... or is it technically the same and makes no difference?
Player 1 has to pass before player 2 gets to arrange his triggers and put them on the stack.

Max mtg wrote:As for me, it looks easier to implement a solution when player's remembered triggers are be added to the stack at the moment he/she gains priority.
Lucky you :)
gos
 
Posts: 4369
Joined: 03 Mar 2011, 15:21
Location: Reykjavík, Iceland
Has thanked: 231 times
Been thanked: 232 times

Re: Born of the Gods Spoiler Season

Postby Sloth » 31 Jan 2014, 10:00

gos wrote:
As far as I can tell this means:
1) "At the beginning of your upkeep"-triggers go on the stack before triggers from the untap phase.
2) The active player gets the opportunity to cast spells and activate abilities before other players' untap phase-triggers even go on the stack.
I think both of these assumptions are wrong.

All untap and upkeep triggers go on the stack at the beginning of upkeep ordered by the controller in APNAP order, then the active player has priority.
User avatar
Sloth
Programmer
 
Posts: 3498
Joined: 23 Jun 2009, 19:40
Has thanked: 125 times
Been thanked: 507 times

Re: Born of the Gods Spoiler Season

Postby Max mtg » 31 Jan 2014, 11:01

@Sloth,
What does that 'lucky you' mean? (I just don't understand how to translate if from English for myself)

Speaking of upkeep triggers, may a player put them into stack in any order (say, have a werewolf transfrom after God-Favored General add 2 tokens)? Of they have to put into stack firstly triggers from untap phase and only then the ones from upkeep?

PS: I will not be able to implement anything until tomorrow anyway. So I'm not that lucky.
Single class for single responsibility.
Max mtg
Programmer
 
Posts: 1997
Joined: 02 Jul 2011, 14:26
Has thanked: 173 times
Been thanked: 334 times

Re: Born of the Gods Spoiler Season

Postby friarsol » 31 Jan 2014, 14:07

I re-reported this issue at the beginning of the spoiler season when it was announced untapped triggers were a thing. What Sloth said is exactly right. If a trigger would happen during a phase where no players would receive priority, those triggers are "held" until a player could receive priority.

This basically translates into: "All triggers that happen during the untap phase and all triggers that happen at beginning of upkeep need to be added to the stack at the time the upkeep ones normally do: at the beginning of the upkeep. Follow normal APNAP rules for ordering."

I had that portion of it working a few months back, but I ran into an issue where for some reason upkeep triggers were triggering twice. Somehow during the adding of triggers, the trigger was firing again somewhere in the stack trace, and thus being added twice. Between Hollowsage and the scripted Inspired cards, there should be enough untap triggers for you to see what's going on. Watch carefully and you'll see that the trigger goes off during the untap phase (trigger on the stack, input box says Untap). The whole paying mana part doesn't actually make a difference, if those cards trigger in the Upkeep without the normal Upkeep triggers being messed up. (I believe Blastoderm was the card I noticed causing me issues, but I'm sure there was probably others)
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Born of the Gods Spoiler Season

Postby gos » 31 Jan 2014, 14:08

Max mtg wrote:What does that 'lucky you' mean? (I just don't understand how to translate if from English for myself)
It means that what I thought was the correct interpretation fitted with the implementation that looked easier to you :)
gos
 
Posts: 4369
Joined: 03 Mar 2011, 15:21
Location: Reykjavík, Iceland
Has thanked: 231 times
Been thanked: 232 times

Re: Born of the Gods Spoiler Season

Postby gos » 31 Jan 2014, 14:11

Sloth wrote:
gos wrote:
As far as I can tell this means:
1) "At the beginning of your upkeep"-triggers go on the stack before triggers from the untap phase.
2) The active player gets the opportunity to cast spells and activate abilities before other players' untap phase-triggers even go on the stack.
I think both of these assumptions are wrong.

All untap and upkeep triggers go on the stack at the beginning of upkeep ordered by the controller in APNAP order, then the active player has priority.
I agree. I misread the first time through.
gos
 
Posts: 4369
Joined: 03 Mar 2011, 15:21
Location: Reykjavík, Iceland
Has thanked: 231 times
Been thanked: 232 times

Re: Born of the Gods Spoiler Season

Postby Max mtg » 31 Jan 2014, 14:57

sol, do not worry.
We can release another beta in a week with this issue fixed and that "you may pay 2{C}" cards properly scripted.

So, shall I take this task or anyone else wants to adjust the moments when "order simultaneous stack entries" is called?
Single class for single responsibility.
Max mtg
Programmer
 
Posts: 1997
Joined: 02 Jul 2011, 14:26
Has thanked: 173 times
Been thanked: 334 times

Re: Born of the Gods Spoiler Season

Postby friarsol » 31 Jan 2014, 15:21

Max mtg wrote:sol, do not worry.
We can release another beta in a week with this issue fixed and that "you may pay 2{C}" cards properly scripted.

So, shall I take this task or anyone else wants to adjust the moments when "order simultaneous stack entries" is called?
I wasn't particularly worried. Just stating a fact. Forge has had this bug for a while now, it's just much more prominent now that more cards have untapping triggers. You can give it a shot. Order simultaneous may get bypassed if there is only one trigger right now, (I can't remember how it's organized)
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Born of the Gods Spoiler Season

Postby Max mtg » 31 Jan 2014, 21:52

Ok, I've commited very simple and naive change to when players may arrange their abilities to be added to stack: r24585
Are there any bugs with this solution?
Single class for single responsibility.
Max mtg
Programmer
 
Posts: 1997
Joined: 02 Jul 2011, 14:26
Has thanked: 173 times
Been thanked: 334 times

Re: Born of the Gods Spoiler Season

Postby friarsol » 31 Jan 2014, 23:04

Max mtg wrote:Ok, I've commited very simple and naive change to when players may arrange their abilities to be added to stack: r24585
Are there any bugs with this solution?
Here's the first one I've found:

AI has Quest Landfall deck with Plated Geopede, Glazing Gradeheart and Sunspring Expedition. I cast Path of Exile on the Gradeheart. As it resolves, AI gets a land, but the two triggers (Geopede and Expedition) don't appear to be on the stack.

Edit: Same deck for AI. I just got a hang. AI played and sacrificed a Terramorphic Expanse. No triggers on the stack after the land was played (before it could be sacrificed). The Input panel claims that I have Priority, but pressing OK doesn't do anything. He has 5 Landfall cards in play. (None of which have targeting choices, so it shouldn't matter what they are).
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Born of the Gods Spoiler Season

Postby Max mtg » 31 Jan 2014, 23:35

Sol, please be more specific about that Landfall AI deck - I'm looking now at "Quest opponent decks" in constructed match lobby and have no clue at all which deck is that 'landfall' one.

yet, the problem seems to be relatively clear: phase should not end when players may have priority and there are unordered simultaneous stack entries.

Commited a changeset to fix that,...
yet I think of another alogrithm: when a player releases priority, stack is being checked - and if there are entries to arrange, he has to order them and then pass again?
Single class for single responsibility.
Max mtg
Programmer
 
Posts: 1997
Joined: 02 Jul 2011, 14:26
Has thanked: 173 times
Been thanked: 334 times

Re: Born of the Gods Spoiler Season

Postby friarsol » 01 Feb 2014, 00:08

Max mtg wrote:Sol, please be more specific about that Landfall AI deck - I'm looking now at "Quest opponent decks" in constructed match lobby and have no clue at all which deck is that 'landfall' one.
I was very clear about what cards were in play. What difference should it make which deck it was? I had to force close, so it's not like I had the name when I was posting.

Edit: That specific Landfall happens to be Aragorn. In Quest mode, they actually provide descriptions as opposed to constructed mode.
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Born of the Gods Spoiler Season

Postby friarsol » 01 Feb 2014, 01:12

The one I reported didn't really have anything to do with the phase ending. A player shouldn't even gain priority if there are triggers that need to go on the stack. And no trigger should occur after a player passes priority.

Additionally, The Terramorphic Expanse + Landfall doesn't appear to be resolved. It was all in Main1.

Play a land (which should cause triggers to happen and be added to the stack)
Sacrifice that land (before the above triggers went on the stack)
Then the triggers finally go on the stack on top of the Land ability.

AI has two Landfall cards out (with Aragorn 1), plays and sacs the Expanse. But the triggers should be on the stack before he's able to Sacrifice it.
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Born of the Gods Spoiler Season

Postby Marek14 » 01 Feb 2014, 07:56

By the way, talking about triggers: if you trigger something during paying a mana cost for spell or ability (like Forbidden Orchard or City of Brass), the trigger should resolve before that spell or ability, not after, as it would go on stack only after it's on stack.
Marek14
Tester
 
Posts: 2761
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 297 times

Re: Born of the Gods Spoiler Season

Postby friarsol » 01 Feb 2014, 13:46

Marek14 wrote:By the way, talking about triggers: if you trigger something during paying a mana cost for spell or ability (like Forbidden Orchard or City of Brass), the trigger should resolve before that spell or ability, not after, as it would go on stack only after it's on stack.
Yea I had this working a year or so ago, and someone broke it, but I could never figure out how it got broken.
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

PreviousNext

Return to Developer's Corner

Who is online

Users browsing this forum: No registered users and 52 guests


Who is online

In total there are 52 users online :: 0 registered, 0 hidden and 52 guests (based on users active over the past 10 minutes)
Most users ever online was 4143 on 23 Jan 2024, 08:21

Users browsing this forum: No registered users and 52 guests

Login Form