Comment 23 for bug 1348800

Revision history for this message
SirVer (sirver) wrote :

> Would you like to take values like that into trunk? I committed them to the linked branch.

Yes, your ideas all sound very reasonable. But I'd like to propose another model. Could you run that through your test case too (you need to build the branch, I changed the model slightly too). Maybe you can apply the tweaks you did to this one too, I updated my github.

The idea here was to optimize not only the trees, but also the weights in the model and the terrains to approximate b18 as closely as possible. This optimization ran a long time (~ 2h) but converges to a small residual value of 0.7, so it should be very close to b18 in behavior.

I put the script I used to optimize here:
https://code.launchpad.net/~widelands-dev/widelands/more_trees

And the branch that contains the values is here:
https://github.com/SirVer/wl_tree_parameter_tuning

Teppo, I wait for your feedback and then we decide which way we want to put into trunk, sounds okay?