Forge version 1.5.6
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
Re: Forge version 1.5.6
by lujo » 24 Nov 2013, 00:09
I got one more bug (sorry for being clueless as to where to report it) - Mercy Killing kills someting and puts an ammount of tokens into play equal to the killed creature's power. But it doesn't take into account if the power had been modified. I'm not sure if this is the right interaction.
---
My Shandalar deck pack folder is avaliable here:Dropbox
Leave feedback on particular decks here: Google doc
Ask for instructions, give feedback and complaints here: Thread
My Shandalar deck pack folder is avaliable here:Dropbox
Leave feedback on particular decks here: Google doc
Ask for instructions, give feedback and complaints here: Thread
Re: Forge version 1.5.6
by Chris H. » 24 Nov 2013, 00:52
lujo wrote:I got one more bug (sorry for being clueless as to where to report it) - Mercy Killing kills someting and puts an ammount of tokens into play equal to the killed creature's power. But it doesn't take into account if the power had been modified. I'm not sure if this is the right interaction.
Read number five in the Forge Frequently Asked Questions.
-
Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: Forge version 1.5.6
by Max mtg » 24 Nov 2013, 07:33
I've moved some packages around - this is to make an easier translation of related classes to game module (less files will be affected)
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: Forge version 1.5.6
by friarsol » 25 Nov 2013, 03:51
I think this looks great, but it seems like we need to get reminder text for the rest of the abilities, since not all abilities have it. (I would offer to compile a list, but it looks like I won't have any time until Sunday, if noone has done it by then I'll sift through them).drdev wrote:I just added the "Hide Reminder Text" setting to the Graphic preference section. However, I chose to default that setting to "off" (so reminder text still displays by default) so as not to change existing behavior.squee1968 wrote:I would like that setting. Also, IMHO, I think the default setting should be set to "off", as the panel would be pretty crowded for some cards, especially for those who make that panel smaller than the default size, making the text harder to read.
The few that I've noticed are:
Miracle
Ripple
Exalted
Fading (I bet Vanishing as well)
Flashback
Totem Armor
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Forge version 1.5.6
by Chris H. » 26 Nov 2013, 16:14
The snapshot builds from the last several days will not launch on my Macintosh computer. Forge freezes while it tries to read in the card files.
Has anyone else noticed this problem? I would like to see this fixed before the next beta release.
Has anyone else noticed this problem? I would like to see this fixed before the next beta release.
-
Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: Forge version 1.5.6
by Max mtg » 26 Nov 2013, 17:40
That's my bad. Fixed in 23805.Chris H. wrote:The snapshot builds from the last several days will not launch on my Macintosh computer. Forge freezes while it tries to read in the card files.
Has anyone else noticed this problem? I would like to see this fixed before the next beta release.
By the way, Chris, how do you like the new screen for constructed games?
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: Forge version 1.5.6
by Chris H. » 26 Nov 2013, 20:59
I updated my local copy to rev 23805. I can launch forge from within Eclipse.
I did another build and deploy and now get a different type of freeze when launching the snapshot. My terminal window states:
And I now get a different message in the progress bar:
I did another build and deploy and now get a different type of freeze when launching the snapshot. My terminal window states:
(ThreadUtil first call): Running on a machine with 2 cpu core(s)
Exception in thread "main" java.lang.ArithmeticException: / by zero
at forge.CardStorageReader.makeTaskListForFiles(CardStorageReader.java:277)
at forge.CardStorageReader.loadCards(CardStorageReader.java:177)
at forge.StaticData.<init>(StaticData.java:41)
at forge.Singletons.initializeOnce(Singletons.java:79)
at forge.view.Main.main(Main.java:40)
And I now get a different message in the progress bar:
-
Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: Forge version 1.5.6
by Chris H. » 26 Nov 2013, 21:04
Max mtg wrote:By the way, Chris, how do you like the new screen for constructed games?
I have been trying to get use to the new constructed mode screen. I now find that I can figure it out without any problem. I assume that most people will adjust with time.
If we get too many complaints I guess that we could revert the constructed mode back to the old setup. We could then add in this recent change to the variant mode section.
-
Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: Forge version 1.5.6
by drdev » 26 Nov 2013, 23:31
I'd be interested at taking a stab at designing a nicer and easier to use game launch screen. Something more like those StarCraft screenshots somebody posted. Ideally I'd like to merge all the Variant launch screens into the Constructed launch screen as part of this. How does that sound?
- drdev
- Programmer
- Posts: 1958
- Joined: 27 Jul 2013, 02:07
- Has thanked: 189 times
- Been thanked: 565 times
Re: Forge version 1.5.6
by Hellfish » 27 Nov 2013, 05:12
Re: constructed home: I'm tinkering with creating a version of DeckLister for players for this purpose, it's just slower going than I thought, mainly due to work.
Merging the screens sounds... intriguing.
Would certainly eliminate a fair bit of code duplication.
Merging the screens sounds... intriguing.
Would certainly eliminate a fair bit of code duplication.
So now you're
Screaming for the blood of the cookie monster
Evil puppet demon of obesity
Time to change the tune of his fearful ballad
C is for "Lettuce," that's good enough for me
Screaming for the blood of the cookie monster
Evil puppet demon of obesity
Time to change the tune of his fearful ballad
C is for "Lettuce," that's good enough for me
-
Hellfish - Programmer
- Posts: 1297
- Joined: 07 Jun 2009, 10:41
- Location: South of the Pumphouse
- Has thanked: 110 times
- Been thanked: 169 times
Re: Forge version 1.5.6
by Agetian » 29 Nov 2013, 03:26
A little something I wanted to discuss regarding r23810 ("Don't show Command Zone if game type doesn't need it"). Won't this break cards like Circles of Protection and Nafs Asp whose effects are shown in command zone even in standard Constructed game? (or has the latter changed already so it's a non-issue?)
- Agetian
- Agetian
- Agetian
- Programmer
- Posts: 3472
- Joined: 14 Mar 2011, 05:58
- Has thanked: 677 times
- Been thanked: 561 times
Re: Forge version 1.5.6
by Max mtg » 29 Nov 2013, 04:07
I'd like to see active effects somewhere during constructed games as well, and command zone was quite a good option.
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: Forge version 1.5.6
by drdev » 29 Nov 2013, 04:59
I did not realize such effects would end up in the Command zone. I definitely intend to rework how the command zone (and other zones) are shown or accessed in the future, but for now I'll revert that change to avoid breaking existing behavior.
EDIT: Fixed in r23820. Note that if you saved a match layout with the command panes hidden from the now reverted change, you'll need to either manually add them back to the layout file or reset your match layout to see them again. This won't affect any end users since they never saw this change.
EDIT: Fixed in r23820. Note that if you saved a match layout with the command panes hidden from the now reverted change, you'll need to either manually add them back to the layout file or reset your match layout to see them again. This won't affect any end users since they never saw this change.
- drdev
- Programmer
- Posts: 1958
- Joined: 27 Jul 2013, 02:07
- Has thanked: 189 times
- Been thanked: 565 times
28 posts
• Page 2 of 2 • 1, 2
Who is online
Users browsing this forum: No registered users and 44 guests