Aether Revolt Spoiler Season
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
Re: Aether Revolt Spoiler Season
by friarsol » 05 Jan 2017, 18:06
If they seem to work, people can just upload them to the upcoming folder. The more eyes the better.Agetian wrote:Here's an archive with Marek's submitted cards so far split into individual files.
- Agetian
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Aether Revolt Spoiler Season
by Agetian » 05 Jan 2017, 19:19
I haven't tested them in the game yet, but generally the scripts look fine, so I guess I'll commit them to the "upcoming" folder and then let's see if people can identify any issues and bugs in them, right?
- Agetian
- Agetian
- Agetian
- Programmer
- Posts: 3471
- Joined: 14 Mar 2011, 05:58
- Has thanked: 676 times
- Been thanked: 561 times
Re: Aether Revolt Spoiler Season
by friarsol » 05 Jan 2017, 19:47
As long as they don't crash and pass the smell test I think I'd be ok with that. The upcoming folder is basically a holding area for unreleased cards.Agetian wrote:I haven't tested them in the game yet, but generally the scripts look fine, so I guess I'll commit them to the "upcoming" folder and then let's see if people can identify any issues and bugs in them, right?
- Agetian
We should just make sure to create a blank editions file for Aether Revolt so all the cards get assigned into it.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Aether Revolt Spoiler Season
by Agetian » 05 Jan 2017, 19:58
I added a small placeholder editions file and tested the cards in the deck editor, they don't crash the game when their script is loaded and when they are visualized in the deck editor. Haven't tested in an actual game yet.
- Agetian
- Agetian
- Agetian
- Programmer
- Posts: 3471
- Joined: 14 Mar 2011, 05:58
- Has thanked: 676 times
- Been thanked: 561 times
Re: Aether Revolt Spoiler Season
by Agetian » 06 Jan 2017, 06:54
I updated Sun Titan to the Secondary syntax.Marek14 wrote:Some notes: I noticed that Sun Titan has split triggers, one for ETB, one for attacking, with different wordings, instead of the "Secondary" syntax that seems to be standard.
Also, some abilities that animate artifacts into artifact creatures just add "Creature" instead of "Artifact,Creature". This matters in cases where the permanent got its artifact type through a temporary effect like Liquimetal Coating.
Also, updated some of the effects I could find with "noncreature artifact becomes an artifact creature", please let me know if that's what you meant.
- Agetian
- Agetian
- Programmer
- Posts: 3471
- Joined: 14 Mar 2011, 05:58
- Has thanked: 676 times
- Been thanked: 561 times
Re: Aether Revolt Spoiler Season
by Agetian » 06 Jan 2017, 08:37
I coded Kari Zev, Skyship Raider and Kari Zev's Expertise today, both are added to upcoming.
- Agetian
- Agetian
- Agetian
- Programmer
- Posts: 3471
- Joined: 14 Mar 2011, 05:58
- Has thanked: 676 times
- Been thanked: 561 times
Re: Aether Revolt Spoiler Season
by Marek14 » 07 Jan 2017, 10:03
OK, I plan to go through the set today and code what I can. Here's the first batch:
Please check that Ajani's Aid is working correctly, it's quite complicated.
Please check that Ajani's Aid is working correctly, it's quite complicated.
- Attachments
-
- aer01.zip
- (6.17 KiB) Downloaded 185 times
Re: Aether Revolt Spoiler Season
by Hanmac » 07 Jan 2017, 10:39
about new cards:
Scrounging Bandar - MoveCounter currently does not support "Amount$ Any" might be helpful for that. (i know its possible to use ChooseNumber before, but i think its better if MoveCounter can do that itself)
Maulfist Revolutionary - for that is probably doable with a loop, but i think a New Effect might be the right choice. (so it can have special AI coded there)
Scrounging Bandar - MoveCounter currently does not support "Amount$ Any" might be helpful for that. (i know its possible to use ChooseNumber before, but i think its better if MoveCounter can do that itself)
Maulfist Revolutionary - for that is probably doable with a loop, but i think a New Effect might be the right choice. (so it can have special AI coded there)
Re: Aether Revolt Spoiler Season
by Marek14 » 07 Jan 2017, 11:19
Second batch.
Question -- I'm using Ceaseless Searblades to code Crackdown Construct, but the code of Ceaseless Searblades seems weird. It has an AbilityCast trigger and TapsForMana trigger, but does it cover mana abilities that don't require tapping (like Manaforge Cinder)? Also, it probably needs a Secondary$ clause.
Question -- I'm using Ceaseless Searblades to code Crackdown Construct, but the code of Ceaseless Searblades seems weird. It has an AbilityCast trigger and TapsForMana trigger, but does it cover mana abilities that don't require tapping (like Manaforge Cinder)? Also, it probably needs a Secondary$ clause.
- Attachments
-
- aer02.zip
- (4.23 KiB) Downloaded 193 times
Last edited by Marek14 on 07 Jan 2017, 12:05, edited 1 time in total.
Re: Aether Revolt Spoiler Season
by Hanmac » 07 Jan 2017, 11:24
Marek14: looks good so far, but Alley Evasion, Charm Spells does not need SpellDescription, its auto generated and is the original one is ignored.
EDIT: i added Revolt similar to all other conditions like Hellbent or FatefulHour
currently there is no way to calculate it. i wanted to make it similar than Morbid, but it didn't work as i wanted, so i add it in GameAction#changeZone
EDIT: i added Revolt similar to all other conditions like Hellbent or FatefulHour
currently there is no way to calculate it. i wanted to make it similar than Morbid, but it didn't work as i wanted, so i add it in GameAction#changeZone
Last edited by Hanmac on 07 Jan 2017, 11:33, edited 1 time in total.
Re: Aether Revolt Spoiler Season
by Agetian » 07 Jan 2017, 11:30
Hmm, Ajani's Aid does not work correctly, it does the search (at least in the library, anyway) but doesn't let you choose Ajani, Valiant Protector and put it into play afterwards :/Marek14 wrote:OK, I plan to go through the set today and code what I can. Here's the first batch:
Please check that Ajani's Aid is working correctly, it's quite complicated.
- Agetian
- Agetian
- Programmer
- Posts: 3471
- Joined: 14 Mar 2011, 05:58
- Has thanked: 676 times
- Been thanked: 561 times
Re: Aether Revolt Spoiler Season
by Marek14 » 07 Jan 2017, 12:08
Hm, I took that part from Liberating Combustion. I thought if there will be problems, it will be in the damage prevention part...Agetian wrote:Hmm, Ajani's Aid does not work correctly, it does the search (at least in the library, anyway) but doesn't let you choose Ajani, Valiant Protector and put it into play afterwards :/Marek14 wrote:OK, I plan to go through the set today and code what I can. Here's the first batch:
Please check that Ajani's Aid is working correctly, it's quite complicated.
- Agetian
Hanmac: So, I can code cards with Revolt wording, they just won't work for now? In that case, can I just code cards with improvise by adding K:Improvise line?
EDIT: Trying to code Daredevil Dragster with help of Clockwork Avian - is there a better condition than "Card.blocking"? For example, if Clockwork Avian regenerates while blocking, it will be removed from combat, but it should still count as having blocked. Conversely, if something puts it onto the battlefield blocking, it should not trigger because it wasn't declared as a blocker. Could you add blockedThisCombat condition similar to attackedThisCombat?
Re: Aether Revolt Spoiler Season
by Hanmac » 07 Jan 2017, 12:26
Revolt is coded so it should just work for new cards. (Like Hellbent) Improvise is not coded yet so it would not work.
For the "blockedThisCombat" i will see what I can do.
For the "blockedThisCombat" i will see what I can do.
Re: Aether Revolt Spoiler Season
by Marek14 » 07 Jan 2017, 12:46
Third batch.
- Attachments
-
- aer03.zip
- (4.56 KiB) Downloaded 175 times
Re: Aether Revolt Spoiler Season
by Agetian » 07 Jan 2017, 13:02
Ah, found out what was wrong, turns out the name had to be specified with a ";" (semicolon) instead of a colon, I fixed that part, but sadly the damage prevention part does not work - first of all, looks like the activation cost is wrong ( instead of "sacrifice Ajani's Aid" ), but even if you pay, it lets you choose the card but it does not prevent combat damage from the chosen card :/ Maybe it can be written using ChooseSource like Circles of Protection and stuff?Marek14 wrote:Hm, I took that part from Liberating Combustion. I thought if there will be problems, it will be in the damage prevention part...
- Agetian
- Agetian
- Programmer
- Posts: 3471
- Joined: 14 Mar 2011, 05:58
- Has thanked: 676 times
- Been thanked: 561 times
Who is online
Users browsing this forum: No registered users and 14 guests