Forge version 1.2.1
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins
64 posts
• Page 3 of 5 • 1, 2, 3, 4, 5
Re: Forge version 1.2.1
by Sloth » 12 Jan 2012, 14:30
That was the problem. Thank you Hellfish. Must have been some problem with ignore lists. The skins weren't updated properly.Hellfish wrote:That's... odd. Is the logo included on the sprite.png for your default skin?
-

Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: Forge version 1.2.1
by Doublestrike » 13 Jan 2012, 00:04
It looks like your sprites aren't updated. I added the logo and start button to each skin sprite; as a result they became taller. That error is from a "getSubImage()" method; fairly self explanatory (the coords provided are outside the image bounds).Sloth wrote:I get this error...
[EDIT] Ah yeah, didn't see the final post at the top of this page. Nailed it already, I see.
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.1
by Chris H. » 13 Jan 2012, 03:21
I am calling it a night and will check back in tomorrow.
I would like to thank Slapshot and Doublestrike for their efforts to fix the memory issues noticed with 1.2.0.
I will release forge 1.2.1 tomorrow unless someone feels that we should devote this upcoming weekend to making a few more adjustments.
I would like to thank Slapshot and Doublestrike for their efforts to fix the memory issues noticed with 1.2.0.
I will release forge 1.2.1 tomorrow unless someone feels that we should devote this upcoming weekend to making a few more adjustments.
-

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.2.1
by slapshot5 » 13 Jan 2012, 06:18
Ok. I've updated the Guava dependency to 11.0.1 since it contains fixes for Caches without a removalListener (which is us). Also, tried to clear out the cache at the end of matches.
It'd be nice to do a couple nightly builds with these changes to see if people have any better experience with long Quest sessions. But if you want to release tomorrow, go for it. It certainly didn't hurt anything memory-wise.
-slapshot5
It'd be nice to do a couple nightly builds with these changes to see if people have any better experience with long Quest sessions. But if you want to release tomorrow, go for it. It certainly didn't hurt anything memory-wise.
-slapshot5
- slapshot5
- Programmer
- Posts: 1391
- Joined: 03 Jan 2010, 17:47
- Location: Mac OS X
- Has thanked: 25 times
- Been thanked: 68 times
Re: Forge version 1.2.1
by Chris H. » 13 Jan 2012, 11:42
`slapshot5 wrote:It'd be nice to do a couple nightly builds with these changes to see if people have any better experience with long Quest sessions. But if you want to release tomorrow, go for it. It certainly didn't hurt anything memory-wise.
I will release a couple of snapshot builds to our testers this weekend and will try to post a few links here and there in our forums.
I will release the 1.2.1 beta on Monday provided that we do not come up with any big issues this weekend.
-

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.2.1
by Sloth » 13 Jan 2012, 13:51
I'm playing against a sliver deck at the moment and not having the actual P/T visible is very annoying. Not being able to check the loyalty counters on planeswalker at a glance is also unpleasant.moomarc wrote:Edit: WRT the mana overlay, I've noticed that there's no mana or P/T overlay if the cards are above a certain size. Is that what you're talking about?
I think that has to be fixed for the beta, otherwise our users will complain for sure.
-

Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: Forge version 1.2.1
by friarsol » 13 Jan 2012, 14:37
I agree, those overlays are the only visual clue we really have in the game and need to be around.Sloth wrote:I'm playing against a sliver deck at the moment and not having the actual P/T visible is very annoying. Not being able to check the loyalty counters on planeswalker at a glance is also unpleasant.moomarc wrote:Edit: WRT the mana overlay, I've noticed that there's no mana or P/T overlay if the cards are above a certain size. Is that what you're talking about?
I think that has to be fixed for the beta, otherwise our users will complain for sure.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Forge version 1.2.1
by Chris H. » 13 Jan 2012, 15:23
At one point the old code would only display the overlays if the card image was below a certain size. I remember that there was some discussion about this and we decided that the overlays should always be visible.
We have a setting to turn off these overlays incase someone feels that they would rather play without them.
We have a setting to turn off these overlays incase someone feels that they would rather play without them.
-

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.2.1
by Sloth » 13 Jan 2012, 15:46
Now i feel like a Hellfish.Chris H. wrote:At one point the old code would only display the overlays if the card image was below a certain size. I remember that there was some discussion about this and we decided that the overlays should always be visible.
We have a setting to turn off these overlays incase someone feels that they would rather play without them.
Thanks Chris. It was just the preferences Doublestrike uploaded.
-

Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: Forge version 1.2.1
by friarsol » 13 Jan 2012, 15:52
The default value should be "on" for this though, otherwise there will be a ton of confusion.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Forge version 1.2.1
by ArsenalNut » 13 Jan 2012, 20:01
Agreed!!! I got caught by this the other night too. Only reason I figured it out was that it had turned off developer mode. Otherwise I wouldn't have thought to look in the settings.friarsol wrote:The default value should be "on" for this though, otherwise there will be a ton of confusion.
So many cards, so little time
-

ArsenalNut - Posts: 512
- Joined: 08 Jul 2011, 03:49
- Has thanked: 27 times
- Been thanked: 121 times
Re: Forge version 1.2.1
by Hellfish » 13 Jan 2012, 20:30
Hey!Sloth wrote:Now i feel like a Hellfish.
I'm not sure if I should feel flattered or insulted.
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.2.1
by Doublestrike » 14 Jan 2012, 00:03
It seems to me that the text/mana overlay should not include the counters or the attack/defend icons - those are pretty critical to the gameplay! Maybe a separate option?
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.1
by Sloth » 14 Jan 2012, 06:55
I thought i read a post from you about this very problem with a Nevermind Edit, but i can't find it anymore. No offense meant.Hellfish wrote:Hey!Sloth wrote:Now i feel like a Hellfish.
I'm not sure if I should feel flattered or insulted.![]()
I don't think they need another option. Who wants to turn off counters or the attack/block icons?Doublestrike wrote:It seems to me that the text/mana overlay should not include the counters or the attack/defend icons - those are pretty critical to the gameplay! Maybe a separate option?
-

Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: Forge version 1.2.1
by Doublestrike » 14 Jan 2012, 07:45
Yeah, that's why I was surprised as the rest of you when they disappeared. So, they should never be hidden, it sounds like.Sloth wrote:I don't think they need another option. Who wants to turn off counters or the attack/block icons?
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
64 posts
• Page 3 of 5 • 1, 2, 3, 4, 5
Who is online
Users browsing this forum: No registered users and 12 guests