Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UST - Unstable card implementation tracking #4233

Open
34 of 53 tasks
spjspj opened this issue Dec 9, 2017 · 16 comments
Open
34 of 53 tasks

UST - Unstable card implementation tracking #4233

spjspj opened this issue Dec 9, 2017 · 16 comments
Labels
cards tracking set Set card implementation tracking

Comments

@spjspj
Copy link
Contributor

spjspj commented Dec 9, 2017

All Sets

https://magic.wizards.com/en/products/unstable

Requires Augment / Host

Not Done Yet:

In Progress :

Done :

@JayDi85
Copy link
Member

JayDi85 commented Dec 9, 2017

!!!WARNING!!!
Don't broke the game and don't add bloating code to game engine. It's a very dangerous set and non legal in any official standards (yeah, I known about some cards legality in commander until mid-January). If game haven't necessary ability, then don't implement it. Technical debt is very bad thing.

I recommend to create check list "possible/unpossible to implement" in xmage.

shot_171209_132701

shot_171209_133150

@spjspj
Copy link
Contributor Author

spjspj commented Dec 9, 2017

These are the ones that are slightly implementable (aka I've taken out ones that have keywords such as watermark, reminder text, 'copy text box' etc. For the 'Host' creature ones I was thinking we could either implement it without the 'Host' part (as I also didn't list the 'Augment' type cards) or just not at all (unless someone wants to implement it).

But Crow Storm is eminently implementable/playable and way cooler than Empty the Warrens. We also have the Star Wars Set as an example of an existing set which is non-standard.

@theelk801
Copy link
Contributor

theelk801 commented Dec 9, 2017

I think Crow Storm is the only one we really need to be honest.

@theelk801
Copy link
Contributor

theelk801 commented Dec 9, 2017

If we want to do die rolls we'll want to add an infrastructure for it to allow for cards that modify them or trigger off of them. I can't say I really want to though, I'd rather start with the voting matters cards from Conspiracy first.

Also counting words in names shouldn't be hard to do.

@JayDi85
Copy link
Member

JayDi85 commented Dec 9, 2017

Die rolls already used before game start. Why you can't use it? Random safe the world.

@theelk801
Copy link
Contributor

Well I just mean so that we can have things which trigger off of and modify the results of die rolls. Just rolling a die is whatever, but making them interact with each other is another matter.

@JayDi85
Copy link
Member

JayDi85 commented Dec 9, 2017

Can you show card's example with modified roll's result?

Found only that (one or two rolls):

  1. Simple roll activating on other trigger (enter to battlefield): Feisty Stegosaurus
    shot_171209_203257
  2. Simple trigger's ability on other trigger (rolling): Ground Pounder
    shot_171209_203436

@theelk801
Copy link
Contributor



@JayDi85 JayDi85 added tracking tasks Todo list for big tasks cards labels Dec 9, 2017
@JayDi85
Copy link
Member

JayDi85 commented Dec 16, 2017

Can't find Dr. Julius Jumblemorph card in todo list. It's already implemented by xmage in Mistform Ultimus card. People want it (by CCG forums).

Dr. Julius Jumblemorph is every creature type (even if this card isn’t on the battlefield).

shot_171216_042442

@spjspj
Copy link
Contributor Author

spjspj commented Dec 16, 2017

^^ Noone's implemented Host/Augment yet so I didn't bother adding it to the list.

@Iwasrunningoutofnamesatthispoint

Its not all about the Host/Augment abillity.
A G/W Changeling makes an awesome commander.

What are the chances to see the other Legendary creatures too? The big idea appears doable to me...

@spjspj
Copy link
Contributor Author

spjspj commented Dec 16, 2017

Yes, the big idea is doable. It's not really a good thing to add a semi-implemented card.

@Iwasrunningoutofnamesatthispoint

I dont see an issue with implementing him this way.
Since we dont have augment cards yet all he will do is to shuffle your lib. Which also seems to be easy to implement.
Nobody will be upset if its done this way :)

(and I will be happy)

@Zzooouhh
Copy link
Contributor

I've made Infinity Elemental, or at least the closest approximation possible within the current engine to my knowledge - its power constantly hovers at Integer.MAX_VALUE and is unchangeable by boost effects unless a "set power" effect (such as Sorceress Queen) overwrites it - this is predominantly intended to prevent integer overflows. I've tested it with boost effects and set P/T effects, please have a look whether I missed anything important: 8bc9ae1

@JayDi85
Copy link
Member

JayDi85 commented Dec 18, 2017

@Zzooouhh
Java can't handle integer overflow at all.

If xmage got overflow "error" then you can't find it and app/server will work at normal but with wrong values. It's a bad thing.

Search xmage code and found Integer.MAX_VALUE in many sources. Try to test Infinity Elemental with single and double strike on Terrifying Presence or another damage prevention card (it have PreventionEffectImpl with max int -- if Infinity Elemental can kill that creature then it's a bug).

@Zzooouhh
Copy link
Contributor

Zzooouhh commented Dec 18, 2017

@JayDi85 Damage prevention correctly negates all the damage Infinity Elemental would deal. What however makes it behave badly is damage value modifying effects (Dictate of the Twin Gods, etc.), double strike damage if the first hit isn't lethal, "gain life equal to this creature's power" type effects, and . All of these are probably solvable by just including an integer overflow check on PlayerImpl.gainLife and PlayerImpl.loseLife (maybe also PlayerImpl.doDamage, which would probably be redundant), I'll try to drum up something similar in the next few commits.

@LevelX2 LevelX2 added tracking set Set card implementation tracking and removed tracking tasks Todo list for big tasks labels Mar 17, 2018
@LevelX2 LevelX2 changed the title UST - Card Implementation Tracker (Tracking issue for Unstable) UST - Unstable card implementation tracking May 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cards tracking set Set card implementation tracking
Projects
None yet
Development

No branches or pull requests

6 participants