Forge version 1.2.1
Post MTG Forge Related Programming Questions Here
	Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins
			64 posts
			 • Page 1 of 5 • 1, 2, 3, 4, 5
		
	
Forge version 1.2.1
 by slapshot5 » 02 Jan 2012, 15:06
by slapshot5 » 02 Jan 2012, 15:06 
Hi all,
How do we want to handle a 1.2.1 release? Do we want to reserve this to fix somewhat serious bugs in 1.2.0 that we can put out another beta fairly soon, or should we start committing big new features (like replacement and any-mana) for a beta 2+ weeks down the road?
And maybe replacement and any-mana are good enough already that this isn't an issue.
-slapshot5
			
		How do we want to handle a 1.2.1 release? Do we want to reserve this to fix somewhat serious bugs in 1.2.0 that we can put out another beta fairly soon, or should we start committing big new features (like replacement and any-mana) for a beta 2+ weeks down the road?
And maybe replacement and any-mana are good enough already that this isn't an issue.
-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. » 02 Jan 2012, 15:26
by Chris H. » 02 Jan 2012, 15:26 
I can pretty much release a 1.2.1 version at any time. Once Doublestrike gets back from his honeymoon he will be able to make a few edits. I can then release a few new versions with only a week or less between releases to get out a few of the more important changes.
Most of the reports that we have gotten so far have been suggestions for further improvements to the UI rather than game-stopping types of bugs.
One person has found that the game on his machine becomes unresponsive during his first match. I have not been able to experience the same problem on my machine. But I may have far more system ram available in my machine.
			
		Most of the reports that we have gotten so far have been suggestions for further improvements to the UI rather than game-stopping types of bugs.
One person has found that the game on his machine becomes unresponsive during his first match. I have not been able to experience the same problem on my machine. But I may have far more system ram available in my machine.
- 
				 
 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 Hellfish » 02 Jan 2012, 15:33
by Hellfish » 02 Jan 2012, 15:33 
Replacement effects are progressing rapidly.Proper "chaining" of replacement effects (for the Reflection cards) is implemented and lifegain,damage and card draw can be replaced.The big problem is how the AI would evaluate them but then again, the current hardcoded replacement effects have that problem as well, AFAICT.
			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 friarsol » 02 Jan 2012, 15:46
by friarsol » 02 Jan 2012, 15:46 
At least if we have a structured replacement effect system the AI could have a chance to add checks for them in the appropriate AFs.
			
		- 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 Hellfish » 02 Jan 2012, 19:45
by Hellfish » 02 Jan 2012, 19:45 
I am in the process of converting the last couple of possible cards to the new system.(EDIT:Treacherous Link and Crumbling Sanctuary remain hardcoded atm. The first because replacement effects can't be statically pumped yet and the second because it's effect can't be scripted yet)After that, I feel confident in committing it right away. Testing so far has been very promising so it should be up to the standard of the current hardcoded cards, if not better, by the next beta either way.
			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 ArsenalNut » 02 Jan 2012, 20:35
by ArsenalNut » 02 Jan 2012, 20:35 
I merged my "Any" mana keyword changes into the trunk today (r12922).  I haven't seen any issues related to my changes over the three days of play testing.
			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 lazylockie » 04 Jan 2012, 06:51
by lazylockie » 04 Jan 2012, 06:51 
Are there still memory leaks on those 1.2.1 Snapshots?
			
		- lazylockie
- Posts: 508
- Joined: 13 Jul 2010, 22:44
- Has thanked: 74 times
- Been thanked: 15 times
Re: Forge version 1.2.1
 by slapshot5 » 04 Jan 2012, 14:11
by slapshot5 » 04 Jan 2012, 14:11 
Don't know for sure, but I don't think there have been any fixes related to these kinds of things yet. So, I would guess "yes" is the answer to your question.lazylockie wrote:Are there still memory leaks on those 1.2.1 Snapshots?
-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. » 10 Jan 2012, 15:30
by Chris H. » 10 Jan 2012, 15:30 
Yesterday, I played several matches using yesterday's snapshot build. I noticed that the memory being used steadily increased during the games and matches.
At the end of the match I went back to the home screen and noticed that the memory being used did not decrease in size. So it looks like we still have the memory leak.
Now that Doublestrike is back he may be able to address this issue at some point in the future. Do we want to release another beta this Friday or do we want to hold off a few weeks to see if we can get the two biggest bugs fixed before the 1.2.1 release.
I think that the two biggest bugs at this time are:
1) the memory leak.
2) the thread issue which leads to the occasional "Buffers have not been created" crash reports.
			
		At the end of the match I went back to the home screen and noticed that the memory being used did not decrease in size. So it looks like we still have the memory leak.
Now that Doublestrike is back he may be able to address this issue at some point in the future. Do we want to release another beta this Friday or do we want to hold off a few weeks to see if we can get the two biggest bugs fixed before the 1.2.1 release.
I think that the two biggest bugs at this time are:
1) the memory leak.
2) the thread issue which leads to the occasional "Buffers have not been created" crash reports.
- 
				 
 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 silly freak » 10 Jan 2012, 18:43
by silly freak » 10 Jan 2012, 18:43 
I have to add that Garbage collection is "on demand", which means that it may just be that you have so much spare memory that nothing was collected yet. I still think it's to investigate though...
			___
where's the "trust me, that will work!" switch for the compiler?
Laterna Magica - blog, forum, project, 2010/09/06 release!
		where's the "trust me, that will work!" switch for the compiler?
Laterna Magica - blog, forum, project, 2010/09/06 release!
- silly freak
- DEVELOPER
- Posts: 598
- Joined: 26 Mar 2009, 07:18
- Location: Vienna, Austria
- Has thanked: 93 times
- Been thanked: 25 times
Re: Forge version 1.2.1
 by Hellfish » 10 Jan 2012, 21:46
by Hellfish » 10 Jan 2012, 21:46 
So we'll just need a System.gc() call in GameAction.newGame()? Huh.
EDIT: Nevermind, I'm a jackass.
			EDIT: Nevermind, I'm a jackass.
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 friarsol » 10 Jan 2012, 22:02
by friarsol » 10 Jan 2012, 22:02 
You've been thanked for being a jackass. (Sorry it's been a long week at work)Hellfish wrote:So we'll just need a System.gc() call in GameAction.newGame()? Huh.
EDIT: Nevermind, I'm a jackass.
- 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. » 11 Jan 2012, 14:33
by Chris H. » 11 Jan 2012, 14:33 
I think that I will go ahead and release 1.2.1 on Friday the 13th. We have a few good fixes already in the code base.
The thread issue and the memory leak/slow dow/garbage collection issue may take awhile to fix. But the fix will provide us with a good reason to release 1.2.2 and 1.2.3.
			
		The thread issue and the memory leak/slow dow/garbage collection issue may take awhile to fix. But the fix will provide us with a good reason to release 1.2.2 and 1.2.3.

- 
				 
 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 Doublestrike » 12 Jan 2012, 00:04
by Doublestrike » 12 Jan 2012, 00:04 
OK thanks Chris.  Yeah, the leak stuff could be a while, or could not, not sure.  I'm making progress though but I've got to leap forward on the learning curve.
I'll do the settings fixes today and tomorrow in between leak testing and whatnot, will try to have that in place for the release.
			I'll do the settings fixes today and tomorrow in between leak testing and whatnot, will try to have that in place for the release.
---
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 Doublestrike » 12 Jan 2012, 04:54
by Doublestrike » 12 Jan 2012, 04:54 
BTW not sure where to put this but I've noticed the "automatic mana selection" funtionality has disappeared.  If I play a 1R spell, pay the 1, then use Lotus Petal or something for the R, I've got to physically select "Red" from the dialog.  Since "White" is first, I've lost a few games because of it 
Before, it would just automagically know I wanted red. It's a global thing since it happens on all multi-mana producing cards. Has anyone else noticed this / is it a problem / what's the rationale behind the change? Hopefully easy to fix?
			
Before, it would just automagically know I wanted red. It's a global thing since it happens on all multi-mana producing cards. Has anyone else noticed this / is it a problem / what's the rationale behind the change? Hopefully easy to fix?
---
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 1 of 5 • 1, 2, 3, 4, 5
		
	
Who is online
Users browsing this forum: No registered users and 35 guests
