Page 4 of 9

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 02 Oct 2016, 02:50
by l33t3rth4nur1111
gos wrote:. Garrulous Sycophant is draining 1 life every turn when no one is the monarch.
Yup. Basically, everybody is the monarch right now, at least for every purpose other than getting to draw a card at BoES.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 02 Oct 2016, 18:57
by lujo
I've just noticed that in FE draft the AI didn't put the rare and bomby Thrull Champion in it's thrull heavy black deck. The champion has SVar:RemRandomDeck:True . This is the only way I can explain this behavior, and the AI should always be playing the card in black decks in FE draft. It's also not noted as a problematic AI card for FE by Xyx in his list.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 02 Oct 2016, 23:49
by Iran
Hello!

Trying to open my Preconstructed Decks, but a receive the following error:

Code: Select all
[spoiler=NullPointerException]
Forge Version:    1.5.56-r-1
Operating System: Windows 7 6.1 amd64
Java Version:     1.8.0_45 Oracle Corporation

java.lang.NullPointerException
   at forge.itemmanager.ColumnDef$42.apply(ColumnDef.java:353)
   at forge.itemmanager.ColumnDef$42.apply(ColumnDef.java:350)
   at forge.itemmanager.views.ItemListView$ItemTableModel.getValueAt(ItemListView.java:756)
   at javax.swing.JTable.getValueAt(JTable.java:2717)
   at javax.swing.JTable.prepareRenderer(JTable.java:5706)
   at javax.swing.plaf.basic.BasicTableUI.paintCell(BasicTableUI.java:2114)
   at javax.swing.plaf.basic.BasicTableUI.paintCells(BasicTableUI.java:2016)
   at javax.swing.plaf.basic.BasicTableUI.paint(BasicTableUI.java:1812)
   at javax.swing.plaf.ComponentUI.update(ComponentUI.java:161)
   at javax.swing.JComponent.paintComponent(JComponent.java:780)
   at forge.toolbox.FSkin$SkinnedTable.paintComponent(FSkin.java:2819)
   at javax.swing.JComponent.paint(JComponent.java:1056)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JViewport.paint(JViewport.java:728)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at forge.toolbox.FScrollPane.paint(FScrollPane.java:96)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JViewport.paint(JViewport.java:728)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JLayeredPane.paint(JLayeredPane.java:586)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JLayeredPane.paint(JLayeredPane.java:586)
   at javax.swing.JComponent.paintChildren(JComponent.java:889)
   at javax.swing.JComponent.paint(JComponent.java:1065)
   at javax.swing.JComponent.paintToOffscreen(JComponent.java:5219)
   at javax.swing.RepaintManager$PaintManager.paintDoubleBuffered(RepaintManager.java:1572)
   at javax.swing.RepaintManager$PaintManager.paint(RepaintManager.java:1495)
   at javax.swing.RepaintManager.paint(RepaintManager.java:1265)
   at javax.swing.JComponent._paintImmediately(JComponent.java:5167)
   at javax.swing.JComponent.paintImmediately(JComponent.java:4978)
   at javax.swing.RepaintManager$4.run(RepaintManager.java:824)
   at javax.swing.RepaintManager$4.run(RepaintManager.java:807)
   at java.security.AccessController.doPrivileged(Native Method)
   at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
   at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:807)
   at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:782)
   at javax.swing.RepaintManager.prePaintDirtyRegions(RepaintManager.java:731)
   at javax.swing.RepaintManager.access$1300(RepaintManager.java:64)
   at javax.swing.RepaintManager$ProcessingRunnable.run(RepaintManager.java:1720)
   at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:311)
   at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:756)
   at java.awt.EventQueue.access$500(EventQueue.java:97)
   at java.awt.EventQueue$3.run(EventQueue.java:709)
   at java.awt.EventQueue$3.run(EventQueue.java:703)
   at java.security.AccessController.doPrivileged(Native Method)
   at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:75)
   at java.awt.EventQueue.dispatchEvent(EventQueue.java:726)
   at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
   at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
   at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
   at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
   at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
   at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)
[/spoiler]
Thanks

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 03 Oct 2016, 13:20
by fsecco
If you reveal a buyback spell via Rashmi, Eternities Crafter's trigger, you can't pay the buyback to play it and put it into your hand, as you should.

Also noticed Queen Marchesa puts tokens regardless who's the Monarch.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 04 Oct 2016, 02:39
by friarsol
lujo wrote:I've just noticed that in FE draft the AI didn't put the rare and bomby Thrull Champion in it's thrull heavy black deck. The champion has SVar:RemRandomDeck:True . This is the only way I can explain this behavior, and the AI should always be playing the card in black decks in FE draft. It's also not noted as a problematic AI card for FE by Xyx in his list.
I think that's where the DeckHints come in. It won't go in a random (generated deck), but it has considerations for Limited Deck Building if the right DeckHints are set for it.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 04 Oct 2016, 14:25
by lujo
Goblin Kites seem to be bugged. If you lose the flip EoT they ask you to "select" a "creature remembered" and only proceede once you've clicked on the crature that's supposed to just automatically die.

Then they seem to display "Remembered: Dwarven Soldier (8)" (for example) even though this creature had died a turn ago. Might not be important, it just looked strange.

Could someone clean them up to work properly?

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 05 Oct 2016, 07:27
by zanizala
First of all congratulation for the excellent job.
I do not play mtg any more (I mean using real cards) due to lack of time (and money).
Forge is a very good alternative.

now the question.

About Energy counters: is there a way to know how many energy counters I have.
I can see them only where I'm using a energy powered ability, for a good strategy should be more useful to have a visible counter.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 05 Oct 2016, 13:27
by friarsol
zanizala wrote:About Energy counters: is there a way to know how many energy counters I have.
I can see them only where I'm using a energy powered ability, for a good strategy should be more useful to have a visible counter.
In this release you have to hover over your player image. In the next release, it will appear next to your life total.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 05 Oct 2016, 16:53
by Mrs Non-Gorilla
I cast a Spinal Parasite with five colors of mana and it was a 5/5 rather than a 4/4. No obvious P/T-affecting effects in sight.

ETA: Also, when casting an Orim's Chant imprinted on an Isochron Scepter, I don't seem to have the option to pay for the Chant's kicker.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 05 Oct 2016, 17:27
by Agetian
Mrs Non-Gorilla wrote:I cast a Spinal Parasite with five colors of mana and it was a 5/5 rather than a 4/4. No obvious P/T-affecting effects in sight.
I can confirm this and I can see the problem (the game engine apparently can't process P/T of -1/-1 for several reasons, starting with the fact that it can't be read correctly at CardFace.java::setPtText and continuing with the fact that certain methods in Card.java (and possibly elsewhere) use P/T value -1 to denote that a value "hasn't been set" (e.g. see Card.java::getLatestPT). Not sure how easy or difficult of a fix this might be, can someone investigate if possible?

- Agetian

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 05 Oct 2016, 17:30
by friarsol
Agetian wrote:Not sure how easy or difficult of a fix this might be, can someone investigate if possible?
Looks like an issue with the card loader. Should be an easy fix.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 05 Oct 2016, 17:38
by Agetian
friarsol wrote:
Agetian wrote:Not sure how easy or difficult of a fix this might be, can someone investigate if possible?
Looks like an issue with the card loader. Should be an easy fix.
I tried tweaking the part of the card loader that processes card faces (CardFace.java) and made it correctly load the card PT as -1/-1, but the card still ends up showing up as 0/0 in hand and ETBing as 0/0 when cast :(

- Agetian

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 05 Oct 2016, 17:42
by friarsol
Agetian wrote:
friarsol wrote:
Agetian wrote:Not sure how easy or difficult of a fix this might be, can someone investigate if possible?
Looks like an issue with the card loader. Should be an easy fix.
I tried tweaking the part of the card loader that processes card faces (CardFace.java) and made it correctly load the card PT as -1/-1, but the card still ends up showing up as 0/0 in hand and ETBing as 0/0 when cast :(

- Agetian
Since it defaults to -1 there's probably checks that call into the card face that check if it's >=0. Default and these checks need to be changed. I'd recommend changing the variable to a nullable (Integer) and check if it's null, and if not grab the intValue

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 05 Oct 2016, 18:12
by Agetian
kk I implemented a fix in r32273, the default P/T value is now Integer.MAX_VALUE (I decided not to try the 'null' approach since that may theoretically provoke numerous NPEs). Basic tests check in (tried a number of standard creatures, X/X, 0/0 and */* creatures, as well as Spinal Parasite, and all appear to work fine), but I'm not sure if something may potentially be broken now... Please test and update if possible/necessary. And thanks for help, Sol! :)

- Agetian

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 06 Oct 2016, 06:05
by Mitlan
There is a problem with Qasali Ambusher (being used by CPUs). I can give a proper bug report (with the template) because the game doesn't crash. When trying to play it by "Mercadia summon" (play free by having certain conditions on the board) it gets retired from stack after switching priorities, then the CPU recast it...and so forth, going into a loop.

Hope my "report" can be usefull 8)