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

2018-02-18 Thread kaputtnik
> But you did add some Ok, i correct my self: I did not add any documentation which were in two places: > Hm, we now have the allowed attributes documented in 2 places. The allowed attributes were already at 2 places in the documentation, before i edited the RST comments. My fault that i didn't

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1669230-lastbastion-finetune into lp:widelands

2018-02-18 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/bug-1669230-lastbastion-finetune into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1669230-lastbastion-finetune/+merge/337083 -- Your team Widel

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

2018-02-18 Thread bunnybot
Continuous integration builds have changed state: Travis build 3206. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/342943879. Appveyor build 3015. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_background

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

2018-02-18 Thread bunnybot
Continuous integration builds have changed state: Travis build 3205. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/342943692. Appveyor build 3014. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_string_fix

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

2018-02-18 Thread TiborB
ad 1) I think the problem is reindeer as an output "ware", we discussed this before, dont we? AI currently knows two categories of productionsites producing non-ware: barracks and carrier trainers (identified as recruitment in aihints). It does not cope with mixed sites yet. ad 2) adding 'bo.i

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

2018-02-18 Thread hessenfarmer
did some testing again fiddling around with the AI hints. Deleting the basic_amount hint for the reindeer farm led to crashes, cause after having achieved basic economy for a frisian tribe the game crashes with the message in stout.txt Max. preciuosness for reindeer farm must not be <= 0, Should

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

2018-02-18 Thread TiborB
I also did small functional change - a kind of max time of scouting -- https://code.launchpad.net/~widelands-dev/widelands/ai_colonization_update/+merge/337913 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_colonization_update. _

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

2018-02-18 Thread TiborB
well, I dont like the idea. But look at this from other angle: Fisher needs near water with fish. Does aquafarm need this as well? -- https://code.launchpad.net/~widelands-dev/widelands/ai_fisher_identification/+merge/337914 Your team Widelands Developers is requested to review the proposed merge

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

2018-02-18 Thread bunnybot
Continuous integration builds have changed state: Travis build 3202. State: failed. Details: https://travis-ci.org/widelands/widelands/builds/342848747. Appveyor build 3011. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_ai_fisher_

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/fh1-maps-multiplayer into lp:widelands

2018-02-18 Thread Klaus Halfmann
Gun: how did you test this on your own? I assume you need a real multiplayer setup to see all the messages? -- https://code.launchpad.net/~widelands-dev/widelands/fh1-maps-multiplayer/+merge/337506 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widel

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

2018-02-18 Thread R M
I restarted the first scenario, old one was a boken save, this time I managed to trigger several extra objectives messages, short of clay, short of bricks etc. It was a bit of a problem remembering what to do as they were not listed as objectives, as was said earlier there are periods where the

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

2018-02-18 Thread GunChleoc
Testing done. This can go in once the NOCOMs have been addressed. -- https://code.launchpad.net/~widelands-dev/widelands/ai_colonization_update/+merge/337913 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_colonization_update. ___

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

2018-02-18 Thread Klaus Halfmann
Review: Needs Fixing Hello Benedikt, I played thfirsian scenario almost upto the end. here some remarks: * I was unable to locate the Portspace at the end of the game and the related building (Weaving Mill and Ship Builder) did not appera in the list of buildings. I assume the AI planted al