[Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-17 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/ai_training_switch into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 -- Your team Widelands Developers is reques

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-17 Thread bunnybot
/_widelands_dev_widelands_ai_training_switch-2387. -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-17 Thread GunChleoc
I'm not too worried about the multiplayer thing. If people go crazy with it, so be it. @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-17 Thread TiborB
Looks good to me... If there will be issues or further requirements we can react to them later... -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-17 Thread kaputtnik
merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-17 Thread GunChleoc
, what do you think? -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-17 Thread GunChleoc
The proposal to merge lp:~widelands-dev/widelands/ai_training_switch into lp:widelands has been updated. Commit Message changed to: The ai training mode is triggered now from the command line. There are 2 switches: --ai_training: Dumps new AI files in home folder and uses higher mutation

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread TiborB
:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread kaputtnik
didn't use this? Is it dependent to the one who is hosting the game? -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread TiborB
Indeed, so what do you prefer? Adding it to single player or modifying help to reflect this... -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread kaputtnik
de.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~wi

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread GunChleoc
/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : wid

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread TiborB
/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread kaputtnik
, depending on FPS. Useful in conjunction with --ai_trainig. Applies only in multiplayer games. ? -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread TiborB
ing_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.lau

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread kaputtnik
ping, imho. -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list:

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread TiborB
/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread TiborB
to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread GunChleoc
/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread GunChleoc
team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread TiborB
This is not related I think... -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread Klaus Halfmann
/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread TiborB
@Gun - but I tested it yesterday... and should be no difference if DEBUG or RELEASE -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread Klaus Halfmann
When wil appvoyer kick in? I would like to fetch the binary form there. OTOH Ill be on tour for some days now. -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-16 Thread GunChleoc
The autospeed doesn't seem to be working any more - I had switched to a release build though. -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread TiborB
:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread TiborB
:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread kaputtnik
ers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread TiborB
://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread kaputtnik
command line description. -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. __

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread TiborB
on many many times -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread TiborB
This is good comment, what about two switches: --ai_training --auto_speed -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread GunChleoc
. Otherwise, it would be better to have multiple options for the command line switch. -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread TiborB
ands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread GunChleoc
>team_number()); > } -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands.

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread TiborB
if (player_->team_number() > 0) { > log("... member of team %d\n", player_->team_number()); > } -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-15 Thread GunChleoc
I have added some comments to the code and gotten rid of a duplicate dynamic cast in interactive base. -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-14 Thread bunnybot
/_widelands_dev_widelands_ai_training_switch-2361. -- https://code.launchpad.net/~widelands-dev/widelands/ai_training_switch/+merge/329008 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_training_switch into lp:widelands

2017-08-14 Thread TiborB
from command line, it has three effects: - dumps new AI files in home folder - activate auto-speed - uses higher mutation ratios There is one NOCOM/question left in the code -- Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands