Page 2 of 3
Re: Forge version 1.3.1

Posted:
12 Nov 2012, 19:48
by moomarc
I don't exactly make huge commits so don't mind either option, but I can see even a two day lockdown being beneficial. Just two days of concentrated bug squashing should make for more stable betas (and lighten the load on Sloth a little

).
Re: Forge version 1.3.1

Posted:
12 Nov 2012, 23:48
by Chris H.
The getting started wiki has a section entitled "Release Schedule and Guidelines".
5. Major changes/submissions to the code base that may be disruptive should be merged into the SVN on the weekend following a beta release. This will give people a chance to find and to fix any problems which may become apparent after a merge. It may be difficult to predict which merges are disruptive and which ones are safe. It could be viewed as something of a crap shoot.
6. Minor new material and bug fixes can be submitted at all times between beta releases. Granted, what is meant to be a minor submission or a minor bug fix could leave the code base in a problematic state. Stuff happens.
7. Major changes to the code base should be discussed with other members of the dev team on the dev topic in advance to submitting the changes to the SVN.
Granted, with the new UI and the new multiplayer projects we have found a number of related issues. It may take us a while to fix them but the project is moving forward. No pain no gain.

Re: Forge version 1.3.1

Posted:
13 Nov 2012, 04:06
by Agetian
Agreed, Chris! And yep, moomarc has a related point too - I think that whenever we get close to release date, we just choose not to submit anything relatively major until after the beta is out, instead concentrating on the bug fixes (as pretty much stated on the wiki). Personally I'm totally OK with it.

- Agetian
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 09:01
by Sloth
I'm still in favor of a beta release soonish. Most of the bugs posted in the 1.3.0 beta topic have been fixed already and it tires me to read them again and again.
If tomorrow is too soon to fix the remaining bugs RumbleBBU has posted (one of which has been around long before the 1.3.0 beta), maybe at the start of next week?
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 12:15
by Chris H.
Sloth wrote:I'm still in favor of a beta release soonish. Most of the bugs posted in the 1.3.0 beta topic have been fixed already and it tires me to read them again and again.
If tomorrow is too soon to fix the remaining bugs RumbleBBU has posted (one of which has been around long before the 1.3.0 beta), maybe at the start of next week?
I think that we should release the 1.3.1 beta on Monday the 19th. I would like to get a beta out fairly soon with the new sound code and hope that a few of the users will be inclined to help us find a few sound files.

It looks like there is a Null and a Concurrent Modification Exception that is causing problems for RumbleBBU's portion of the project. I hope that this will be fixed by Monday.
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 16:23
by moomarc
I think Monday should be good. Hopefully the last of those quest issues will be sorted out, and it will give Agetian a few days to sort out the problem I discovered with the restricted mana for
X being payable with any color from the mana pool.
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 17:15
by Agetian
Agreed, Monday is a good date, I think we should be good by then. The bug with the restricted
X mana pool is fixed, by the way, thanks for catching it!
- Agetian
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 17:35
by moomarc
Agetian wrote:The bug with the restricted
X mana pool is fixed, by the way, thanks for catching it!
- Agetian
Excellent! Thanks Speedy!
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 18:07
by Agetian
moomarc wrote:Agetian wrote:The bug with the restricted
X mana pool is fixed, by the way, thanks for catching it!
- Agetian
Excellent! Thanks Speedy!
Hehe yep, trying to work fast.

Hope nothing else serious shows up (or if it does, I get the chance to fix up before the release). So far, seems to work fine though!

- Agetian
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 18:47
by RumbleBBU
Well, the quest challenge bug is not any worse (if not better either) than before, and as for the quest embark submenu NPE, I could take a look at that if I have time and Max or someone else doesn't beat me to it.
So yes, Monday sounds perfectly good.
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 21:11
by Max mtg
commited some different brains for quest sets unlocks.
A set unlock token is awarded for each 20 wins. They are 'kept' in inventory and can be used at any time.
Number of wins doesn't really matter - might change it to 12 or 60, I don't care for now.
NPE mentioned above does not happen anymore.
As for me releasing a beta tomorrow would be better than doing so on Monday.
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 21:30
by moomarc
I like the idea of the unlock token. Would you still have to pay as well, seeing as you get a starter and booster (or whatever the base pack was)? It's a great idea becuase you don't have to worry about missing your unlock by a handful of credits. Thanks!
Re: Forge version 1.3.1

Posted:
15 Nov 2012, 22:03
by Max mtg
moomarc wrote:I like the idea of the unlock token. Would you still have to pay as well, seeing as you get a starter and booster (or whatever the base pack was)? It's a great idea becuase you don't have to worry about missing your unlock by a handful of credits. Thanks!
It gives the same initial cards as Rumble has designed.
The prices are also unchanged
- Code: Select all
IStorageView<BoosterData> starters = Singletons.getModel().getTournamentPacks();
IStorageView<BoosterData> boosters = Singletons.getModel().getBoosters();
qData.getFormat().unlockSet(unlockedSet.getCode());
List<CardPrinted> cardsWon = new ArrayList<CardPrinted>();
if (starters.contains(unlockedSet.getCode())) {
UnOpenedProduct starter = new UnOpenedProduct(starters.get(unlockedSet.getCode()));
cardsWon.addAll(starter.open());
}
else if (boosters.contains(unlockedSet.getCode())) {
UnOpenedProduct booster = new UnOpenedProduct(boosters.get(unlockedSet.getCode()));
cardsWon.addAll(booster.open());
cardsWon.addAll(booster.open());
cardsWon.addAll(booster.open());
}
qData.getCards().addAllCards(cardsWon);
final CardListViewer cardView = new CardListViewer(unlockedSet.getName(), "You get the following bonus cards:", cardsWon);
cardView.show();
qData.save();
}
- Code: Select all
final ReadPriceList prices = new ReadPriceList();
final Map<String, Integer> mapPrices = prices.getPriceList();
final List<ImmutablePair<CardEdition, Integer>> setPrices = new ArrayList<ImmutablePair<CardEdition,Integer>>();
for (CardEdition ed : getUnlockableEditions(qData)) {
int price = 7500;
if (mapPrices.containsKey(ed.getName() + " Booster Pack")) {
price = Math.max( 50 * mapPrices.get(ed.getName() + " Booster Pack"), 7500 );
}
setPrices.add(ImmutablePair.of(ed, price));
}
Re: Forge version 1.3.1

Posted:
16 Nov 2012, 01:06
by Chris H.
Max mtg wrote:commited some different brains for quest sets unlocks.
A set unlock token is awarded for each 20 wins. They are 'kept' in inventory and can be used at any time.
Number of wins doesn't really matter - might change it to 12 or 60, I don't care for now.
NPE mentioned above does not happen anymore.
As for me releasing a beta tomorrow would be better than doing so on Monday.
Thank you Max, I can release tomorrow if no one comes up with a major problem.

Re: Forge version 1.3.1

Posted:
16 Nov 2012, 03:16
by Max mtg
Looks like the only concurrent modification is a problem 4 now.
Yet, it would be valuable to evaluate or test my code - it might contain new bugs

Why I consider friday a better day for release? Weekend would be a 'code freeze' period otherwise, that leaves less space for global changes (like replace sound and log calls in main code with events thrown into a channel)