Created subversion branch /branches/braids-minimax
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins
10 posts
• Page 1 of 1
Created subversion branch /branches/braids-minimax
by Braids » 22 Jun 2011, 21:58
it is currently based on r9515. i'm trying to get the minimax AI changes into the repository under this branch so it doesn't mess up main line development.
"That is the dumbest thing I've ever seen." --Rob Cashwalker, regarding Innistrad double-sided cards. One of the first times he and I have ever agreed on something. 
-

Braids - Programmer
- Posts: 556
- Joined: 22 Jun 2011, 00:39
- Location: Unknown. Hobby: Driving myself and others to constructive madness.
- Has thanked: 1 time
- Been thanked: 1 time
Re: Created subversion branch /branches/braids-minimax
by Rob Cashwalker » 23 Jun 2011, 13:48
Thanks. How difficult is it to create and maintain branches? Will the non-AI parts of your project be able to sync with the main branch for bug fixes?
The Force will be with you, Always.
-

Rob Cashwalker - Programmer
- Posts: 2167
- Joined: 09 Sep 2008, 15:09
- Location: New York
- Has thanked: 5 times
- Been thanked: 40 times
Re: Created subversion branch /branches/braids-minimax
by mtgrares » 23 Jun 2011, 16:25
Braids, sounds great. I didn't think Forge would ever use minimax...maybe pigs really do fly?
- mtgrares
- DEVELOPER
- Posts: 1352
- Joined: 08 Sep 2008, 22:10
- Has thanked: 3 times
- Been thanked: 12 times
Re: Created subversion branch /branches/braids-minimax
by Braids » 23 Jun 2011, 18:14
you're welcome! the difficulty is proportional to the skill & experience in dealing with version control systems. i have a lot fo both, although not w subversion specifically. i've already created the branch. i don't think we'll need another. the very nature of a branch is to provide a means of structured & tracked synchronization - through merging.Rob Cashwalker wrote:Thanks. How difficult is it to create and maintain branches? Will the non-AI parts of your project be able to sync with the main branch for bug fixes?
the non-AI parts could be merged from the ai branch into the trunk (the main line yall have been using) on a file by file basis. but i don't think that's going to happen very often, though.
maintenance is mostly my doing because instead, i'll be merging the changes in the other direction, from the trunk into the AI branch, perhaps weekly. it involves editing *around* nearly every trunk change by hand. for the most prt, i remove the old code that uses new ai method calls, and i change the incoming (trunk) code to use the new method calls. these changes occur on the braids-minimax branch only. that's how i found out about a lot of card specific code being taken out. i suspect itis going to get old fast for me. i may advocate (and beg and plead) for certain subtle changes to the trunk to make my life easier. especially changing references to certain fields into method calls.
if there's something unobtrusive (for example, a serialVersionUID) that i want on the ai branch and yet the trunk would benefit, i'll probably change the trunk myself and commit it as if it were a bugfix or new card, then merge it outward to the ai branch.
as for the braids-minimax branch, well, it began as a branch off of r9515. i copied in the latest minimax source. i should have committed at that point. but instead i ran svn update, because i thought i was supposed to do that before commiting. i should have specified -r9515 to update. instead it is essentially forcing me to merge all the changes from r9515 to present into the braids-minimax branch!!! i have yet to commit
edit 6/23: add quote and politeness.
Last edited by Braids on 23 Jun 2011, 18:23, edited 1 time in total.
"That is the dumbest thing I've ever seen." --Rob Cashwalker, regarding Innistrad double-sided cards. One of the first times he and I have ever agreed on something. 
-

Braids - Programmer
- Posts: 556
- Joined: 22 Jun 2011, 00:39
- Location: Unknown. Hobby: Driving myself and others to constructive madness.
- Has thanked: 1 time
- Been thanked: 1 time
Re: Created subversion branch /branches/braids-minimax
by Braids » 23 Jun 2011, 18:22
thank you. minimax still might suck. game trees take up a lot of memory. i'm hoping the 1st gen ai can help with that.mtgrares wrote:Braids, sounds great. I didn't think Forge would ever use minimax...maybe pigs really do fly?
right now, i'm aiming to make the game state unit testable and as a side benefit, to make it serializable so people can save in the middle of a game or match.
minimax ai is long term. i have some other wants, like lazy initialization of CardFactory, and loading .DEC or .mwDeck files from the deck editor. i want to build concensus first of course.
"That is the dumbest thing I've ever seen." --Rob Cashwalker, regarding Innistrad double-sided cards. One of the first times he and I have ever agreed on something. 
-

Braids - Programmer
- Posts: 556
- Joined: 22 Jun 2011, 00:39
- Location: Unknown. Hobby: Driving myself and others to constructive madness.
- Has thanked: 1 time
- Been thanked: 1 time
Re: Created subversion branch /branches/braids-minimax
by friarsol » 23 Jun 2011, 18:34
Lazy Initialization of CardFactory would be pretty handy. We probably want to change the Cards.txt files over to use an Oracle Text Field to display card info in the Deck Editor before that happens, otherwise we lose all the benefit of Lazy Initialization there.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Created subversion branch /branches/braids-minimax
by Braids » 23 Jun 2011, 18:43
id' rather do this one step at a time. CardFactory's init was slowing unit testing so much, i edited main.properties to use a different card directory and put only 5 cards in there. i think we'll get a lot of benefit except for the deck editor; we can postpone that.friarsol wrote:Lazy Initialization of CardFactory would be pretty handy. We probably want to change the Cards.txt files over to use an Oracle Text Field to display card info in the Deck Editor before that happens, otherwise we lose all the benefit of Lazy Initialization there.
once i've updated and committed the latest braids-minimax branch **grumble**, i'll see about building concensus for lazy cardfactory and hopefully converting AllZone field access to method calls on the svn trunk "src"... otherwise i'm going to be a very very unhappy merger (merge maker?).
"That is the dumbest thing I've ever seen." --Rob Cashwalker, regarding Innistrad double-sided cards. One of the first times he and I have ever agreed on something. 
-

Braids - Programmer
- Posts: 556
- Joined: 22 Jun 2011, 00:39
- Location: Unknown. Hobby: Driving myself and others to constructive madness.
- Has thanked: 1 time
- Been thanked: 1 time
Re: Created subversion branch /branches/braids-minimax
by friarsol » 23 Jun 2011, 19:08
Fair enough. At the very least it'll speed up the rest of the game aside from the Deck Editor. The Oracle Text is something that has been brought up a fair amount in the last few months, and is bound to happen sooner or later.Braids wrote:id' rather do this one step at a time. CardFactory's init was slowing unit testing so much, i edited main.properties to use a different card directory and put only 5 cards in there. i think we'll get a lot of benefit except for the deck editor; we can postpone that.
Merge maker merge maker, make me a merge, diff me a diff, branch me a branch....Braids wrote:once i've updated and committed the latest braids-minimax branch **grumble**, i'll see about building concensus for lazy cardfactory and hopefully converting AllZone field access to method calls on the svn trunk "src"... otherwise i'm going to be a very very unhappy merger (merge maker?).
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Created subversion branch /branches/braids-minimax
by Chris H. » 23 Jun 2011, 19:27
It might be a good idea to create a separate "Note:" field for the card txt files as this would allow us to remove the bug notes from the text line.
The value in the "Note:" field could then be added to both the text field for in game notice and to the oracle field for the deck editor.
I had done some initial work in this area a few months ago but never got it working and got busy with other areas of the project.
The value in the "Note:" field could then be added to both the text field for in game notice and to the oracle field for the deck editor.
I had done some initial work in this area a few months ago but never got it working and got busy with other areas of the project.
-

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: Created subversion branch /branches/braids-minimax
by Rob Cashwalker » 23 Jun 2011, 19:49
Didn't I mention WAY-back wanting to convert the Text field to actually use the oracle text? Then all the scripts don't have to carry so much extra baggage to populate the text dynamically, just enough to deal with interaction.friarsol wrote:Lazy Initialization of CardFactory would be pretty handy. We probably want to change the Cards.txt files over to use an Oracle Text Field to display card info in the Deck Editor before that happens, otherwise we lose all the benefit of Lazy Initialization there.
The Force will be with you, Always.
-

Rob Cashwalker - Programmer
- Posts: 2167
- Joined: 09 Sep 2008, 15:09
- Location: New York
- Has thanked: 5 times
- Been thanked: 40 times
10 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 20 guests