Board index 
Programs with AI or Rules Enforcement Magic: The Gathering - Duels of the Planeswalkers New MTG Cards and Decks (2010, 2012, 2013, 2014, 2015, Magic Duels) 2014
Programs with AI or Rules Enforcement Magic: The Gathering - Duels of the Planeswalkers New MTG Cards and Decks (2010, 2012, 2013, 2014, 2015, Magic Duels) 2014
Khans of Tarkir (102/254) Last Updated 9/21/14
Moderator: CCGHQ Admins
56 posts
• Page 4 of 4 • 1, 2, 3, 4
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by NeoAnderson » 24 Sep 2014, 00:22
Now i would approach to make a Delve approximation but i am thinking we need to make a choice, because of Delve definition it isn't an additional cost or an alternative cost, but the only way to make it properly works is to implement as alternative cost.
Another way could be to implement as activated ability but in this case we will lose the possibility to use from other zone like exile, graveyard, library.....
Theoretically we should be able to cast Delve + Flashback and i am not sure we can do it.
Honestly i am not sure which could be the best solution...
Another way could be to implement as activated ability but in this case we will lose the possibility to use from other zone like exile, graveyard, library.....
Theoretically we should be able to cast Delve + Flashback and i am not sure we can do it.
Honestly i am not sure which could be the best solution...
- NeoAnderson
- Posts: 914
- Joined: 10 Sep 2013, 07:49
- Has thanked: 18 times
- Been thanked: 139 times
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by Xander9009 » 24 Sep 2014, 01:04
I hadn't yet tested it. I noticed a number of issues but I think I got them all fixed, along with splitting it into two functions. However, I still can't manage a test which confirms a change in cost. How did you manage that Neo? (Note: I AM using the two pieces of code you just posted.) I'm not getting any errors, it but the cost function always return 0. Since you didn't post the code cut up into 2 functions, I'll post my version.
That leads to a useful question: how do you guys debug your code? Because the only way I know to do it is to put in something like GainLife(VariableToDebug). That's really clunky, though. Is there a better way?
EDIT:
NVM. If you've got it, Neo, that's good enough for me. As for which would be better, I would go with whichever is most reliable. Alternative should be acceptable.
That leads to a useful question: how do you guys debug your code? Because the only way I know to do it is to put in something like GainLife(VariableToDebug). That's really clunky, though. Is there a better way?
- | Open
EDIT:
NVM. If you've got it, Neo, that's good enough for me. As for which would be better, I would go with whichever is most reliable. Alternative should be acceptable.
_______________________________
Community Wad - Community Wad Website - How to Help and Report Bugs
Discord: discord.gg/4AXvHzW
Community Wad - Community Wad Website - How to Help and Report Bugs
Discord: discord.gg/4AXvHzW
-
Xander9009 - Programmer
- Posts: 2905
- Joined: 29 Jun 2013, 07:44
- Location: Indiana, United States
- Has thanked: 121 times
- Been thanked: 445 times
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by NeoAnderson » 24 Sep 2014, 06:00
To test my code i am using 2 modded cards who let me know to force many situations, and i use one of them to build custom abilities for particular tests. For this function i used the follow ability :Xander9009 wrote:I hadn't yet tested it. I noticed a number of issues but I think I got them all fixed, along with splitting it into two functions. However, I still can't manage a test which confirms a change in cost. How did you manage that Neo? (Note: I AM using the two pieces of code you just posted.) I'm not getting any errors, it but the cost function always return 0. Since you didn't post the code cut up into 2 functions, I'll post my version.
That leads to a useful question: how do you guys debug your code? Because the only way I know to do it is to put in something like GainLife(VariableToDebug). That's really clunky, though. Is there a better way?
EDIT:
NVM. If you've got it, Neo, that's good enough for me. As for which would be better, I would go with whichever is most reliable. Alternative should be acceptable.
- ABILITY TO CHECK TARGET COST CHANGE | Open
- ABILITY TO CHECK ACTIVE ABILITIES STATUS | Open
2. Before to choose Alternative cost or Activated ability i want to make some tests, to understand if there is a way to take a decision before the cost of the spell is payed when we normally attempt to cast it.
3. I post here also the modified function so if you need you can compare
- UpdateColourlessCostValue | Open
- COSTANTS DECLARATION | Open
- NeoAnderson
- Posts: 914
- Joined: 10 Sep 2013, 07:49
- Has thanked: 18 times
- Been thanked: 139 times
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by Xander9009 » 24 Sep 2014, 13:43
Ironically, I saw Display Message and considered using it. For whatever reason, I figured it was probably only going to work with strings and not numbers and decided against trying it. Anyway, I've got your script put in, but the checking ability is still returning 0 even when it should be returning 1 (and yes, I'm using your fixed version of the manager). I'll mess with it some today, but I'll be mostly focused on getting the tokens ready in the Community Wad (I have about 50 pictures left to do and every single token I could find will be available in a standard format 
).
)._______________________________
Community Wad - Community Wad Website - How to Help and Report Bugs
Discord: discord.gg/4AXvHzW
Community Wad - Community Wad Website - How to Help and Report Bugs
Discord: discord.gg/4AXvHzW
-
Xander9009 - Programmer
- Posts: 2905
- Joined: 29 Jun 2013, 07:44
- Location: Indiana, United States
- Has thanked: 121 times
- Been thanked: 445 times
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by NeoAnderson » 24 Sep 2014, 16:00
1. About the DisplayMessage function you can pass everything you want strings or numbers, or both concatenating the values.Xander9009 wrote:Ironically, I saw Display Message and considered using it. For whatever reason, I figured it was probably only going to work with strings and not numbers and decided against trying it. Anyway, I've got your script put in, but the checking ability is still returning 0 even when it should be returning 1 (and yes, I'm using your fixed version of the manager). I'll mess with it some today, but I'll be mostly focused on getting the tokens ready in the Community Wad (I have about 50 pictures left to do and every single token I could find will be available in a standard format
Example:
- Code: Select all
local value = 1
EffectController():DisplayMessage("The result is : "..value)
- Code: Select all
The result is : 1
3. I would continue the discussion about Delve mechanic into a dedicated topic, so i opened a topic into programming talk, i also made the first approximation, follow the link and let to discess there.
DELVE TOPIC- NeoAnderson
- Posts: 914
- Joined: 10 Sep 2013, 07:49
- Has thanked: 18 times
- Been thanked: 139 times
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by Tejahn » 24 Sep 2014, 19:47
I'm glad to hear of the progress that's been made for Delve. I'll be sure to build a few decks and test the mechanic. Also, I've uploaded the Mardu deck. It's brutally efficient and mulligans very well if needed. You'll always have a high chance of a quick victory.
EDIT: I re-uploaded the Mardu deck to include the Vampire token created by Sorin.
EDIT: I re-uploaded the Mardu deck to include the Vampire token created by Sorin.
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by volrathxp » 27 Sep 2014, 20:30
Your Sarkhan is missing the second half of his emblem (the part about discarding your hand at the end of turn). Right now it has you just drawing 2 additional.
Edit: Also, how are you accounting for the fact that new Sarkhan isn't in the planeswalker name set? Same for Sorin. The PLW Mod functions includes a NameSet function that lists every planeswalker name, to account for specific cards and interactions. New Sarkhan and Sorin will need to be added to this file.
Edit: Also, how are you accounting for the fact that new Sarkhan isn't in the planeswalker name set? Same for Sorin. The PLW Mod functions includes a NameSet function that lists every planeswalker name, to account for specific cards and interactions. New Sarkhan and Sorin will need to be added to this file.
volrathxp
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by Tejahn » 27 Sep 2014, 23:58
Thanks for the report volrathxp! Could you post the code for Sarkhan's emblem? I'll update it with the proper code. Also, if you could do the same for the NameSet functions for the Planeswalkers that would be great.
EDIT: I didn't bother with the NameSet functions because I was waiting on thefiremind to update the Planeswalker Mod to include the new Sarkhan and Sorin.
EDIT: I didn't bother with the NameSet functions because I was waiting on thefiremind to update the Planeswalker Mod to include the new Sarkhan and Sorin.
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by volrathxp » 28 Sep 2014, 02:56
Yeah, I think that's for the best in the long run.Tejahn wrote:Thanks for the report volrathxp! Could you post the code for Sarkhan's emblem? I'll update it with the proper code. Also, if you could do the same for the NameSet functions for the Planeswalkers that would be great.
EDIT: I didn't bother with the NameSet functions because I was waiting on thefiremind to update the Planeswalker Mod to include the new Sarkhan and Sorin.
volrathxp
Re: Khans of Tarkir (102/254) Last Updated 9/21/14
by fallenangle » 28 Sep 2014, 20:33
I got Sarkhan's Emblem to work. Here's the code I used. There's probably a more elegant way to do it, but I'm only a novice coder, so I don't know. At any rate, here's the code for his ultimate:
Edit: I found that I accidentally had an old copy of the PLW.LOL in my functions folder. Deleting it did the trick, and now everything works just as it should. All of the new Sarkhan's and Sorin's abilities work properly, and they add and subtract loyalty counters, use abilities, and go to the graveyard when they have no loyalty counters just as they should. I will try to upload my working, updated PLW.wad to the Planeswalkers topic. That should give people a chance to have all of the Planeswalkers available in one place.
- Code: Select all
<ACTIVATED_ABILITY>
<LOCALISED_TEXT LanguageCode="en-US"><![CDATA[[-6]: You get an emblem with “At the beginning of your draw step, draw two additional cards” and “At the beginning of your end step, discard your hand.”]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="fr-FR"><![CDATA[-6 : Vous gagnez un emblème avec « Au début de votre étape de pioche, piochez deux cartes supplémentaires » et « Au début de votre étape de fin, défaussez-vous de votre main. »]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="es-ES"><![CDATA[–6: Obtienes un emblema con “Al comienzo de tu paso de robar, roba dos cartas adicionales” y “Al comienzo de tu paso final, descarta tu mano”.]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="de-DE"><![CDATA[−6: Du erhältst ein Emblem mit „Ziehe zu Beginn deines Ziehsegments zwei zusätzliche Karten” und „Wirf zu Beginn deines Endsegments alle Karten aus deiner Hand ab.”]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="it-IT"><![CDATA[−6: Ottieni un emblema con “All’inizio della tua acquisizione, pesca due carte addizionali” e “All’inizio della tua sottofase finale, scarta la tua mano”.]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="jp-JA"><![CDATA[－６：あなたは「あなたのドロー・ステップの開始時に、カードを追加で２枚引く。」と「あなたの終了ステップの開始時に、あなたは手札を捨てる。」を持つ紋章を得る。]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="ko-KR"><![CDATA[-6: 당신은 “당신의 뽑기단 시작에, 카드 두 장을 추가로 뽑는다.”와 “당신의 종료단 시작에, 당신의 손을 버린다.”를 가진 휘장을 받는다.]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="ru-RU"><![CDATA[−6: вы получаете эмблему со способностями «В начале вашего шага взятия карты возьмите две дополнительные карты» и «В начале вашего заключительного шага сбросьте вашу руку».]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="pt-BR"><![CDATA[−6: Você ganha um emblema com “No início da sua etapa de compra, compre dois cards adicionais” e “No início da sua etapa final, descarte a sua mão.”]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="zh-CN"><![CDATA[-6：你获得具有「在你的抓牌步骤开始时，额外抓两张牌」与「在你的结束步骤开始时，弃掉你的手牌」的徽记。]]></LOCALISED_TEXT>
<LOCALISED_TEXT LanguageCode="zh-HK"><![CDATA[-6：你獲得具有「在你的抽牌步驟開始時，額外抽兩張牌」與「在你的結束步驟開始時，棄掉你的手牌」的徽記。]]></LOCALISED_TEXT>
<AVAILABILITY sorcery_time="1" />
<COST type="RemoveCountersSelf" amount="6" counter_type="Loyalty" />
<RESOLUTION_TIME_ACTION>
MTG():CreateDelayedTrigger(1, nil)
</RESOLUTION_TIME_ACTION>
<RESOLUTION_TIME_ACTION>
MTG():CreateDelayedTrigger(2, nil)
</RESOLUTION_TIME_ACTION>
</ACTIVATED_ABILITY>
<TRIGGERED_ABILITY resource_id="1">
<CLEANUP>
return EffectController() == nil
</CLEANUP>
<TRIGGER value="BEGINNING_OF_PLAYERS_STEP" simple_qualifier="controller">
return MTG():GetStep() == STEP_UPKEEP
</TRIGGER>
<RESOLUTION_TIME_ACTION>
if TriggerPlayer() ~= nil then
TriggerPlayer():DrawCards(2)
end
</RESOLUTION_TIME_ACTION>
</TRIGGERED_ABILITY>
<TRIGGERED_ABILITY resource_id="2">
<CLEANUP>
return EffectController() == nil
</CLEANUP>
<TRIGGER value="BEGINNING_OF_PLAYERS_STEP" simple_qualifier="controller">
return MTG():GetStep() == STEP_END_OF_TURN
</TRIGGER>
<RESOLUTION_TIME_ACTION>
EffectController():DiscardHand()
</RESOLUTION_TIME_ACTION>
</TRIGGERED_ABILITY>
Edit: I found that I accidentally had an old copy of the PLW.LOL in my functions folder. Deleting it did the trick, and now everything works just as it should. All of the new Sarkhan's and Sorin's abilities work properly, and they add and subtract loyalty counters, use abilities, and go to the graveyard when they have no loyalty counters just as they should. I will try to upload my working, updated PLW.wad to the Planeswalkers topic. That should give people a chance to have all of the Planeswalkers available in one place.
- fallenangle
- Posts: 319
- Joined: 20 Jul 2013, 02:31
- Has thanked: 73 times
- Been thanked: 41 times
56 posts
• Page 4 of 4 • 1, 2, 3, 4
Who is online
Users browsing this forum: No registered users and 14 guests