<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/112747/">http://git.reviewboard.kde.org/r/112747/</a>
     </td>
    </tr>
   </table>
   <br />










<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On October 16th, 2013, 9:26 p.m. UTC, <b>Albert Astals Cid</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
 <thead>
  <tr>
   <th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
    <a href="http://git.reviewboard.kde.org/r/112747/diff/3/?file=200909#file200909line24" style="color: black; font-weight: bold; text-decoration: underline;">players/ai/becai/becai_gui.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 3)

    </span>
   </th>
  </tr>
 </thead>



 
 

 <tbody>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">24</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">PlayerGui</span><span class="p">(</span><span class="n">i18n</span><span class="p">(</span><span class="s">"Becai"</span><span class="p">))</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Is "Becai" something players would recognise? I don't, but I'm not a konquest player either :D</pre>
 </blockquote>



 <p>On October 18th, 2013, 11:51 p.m. UTC, <b>Nemanja Hirsl</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I agree. Who/What might Becai be? For now we have default weak, normal/offensive and hard/defensive players - so maybe a name which would indicate strategy?</pre>
 </blockquote>





 <p>On October 19th, 2013, 1:16 a.m. UTC, <b>Alexander Schuch</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I do not really care for what the GUI displays. We can call it "Becai (sophisticated)", but that does not say much more either. And in my opinion, "normal" and "hard" are misleading as well, as depending on the strategy, normal can be harder to defeat than hard.

Just play a game with plenty of planets, normal versus hard. Normal keeps a smaller defence, so it attacks earlier. This results in normal getting more planets quickly. Due to a much higher ship production then, it can easily conquer hard. - I just tried that with weak versus hard and weak plays about as good as hard does. Out of 10 games it is roughly 1:1 ratio of winning. My setup is 30x22 board with 50 neutral planets and 1 neutral production.

Furthermore, I am developing Becai 2 as well (will take some time, though) and I use Becai as benchmark. How should that AI be called then - "more sophisticated"? Dropping Becai in favour of Becai 2 might not be a good idea either as the gap between default and Becai 2 will be larger then.

I personally like the idea to give distinct unique names to AI implementations, as done for C-evo, OpenTTD and other empire games. It just is not possible to summarise decision making (a strategy) in one or two words. That's why we later on should add a description field where the strategy of the selected player controller is shown, in plain easy English.

Maybe at least use "PlayerGui(i18nc("A distinct unique name", "Becai"))" to not astonish translators about the term.</pre>
 </blockquote>





 <p>On October 19th, 2013, 5:57 p.m. UTC, <b>Albert Astals Cid</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Well, if you can't summarize Becai in two words, then you probably should edit the documentation and explain there what's it about?</pre>
 </blockquote>





 <p>On October 19th, 2013, 5:57 p.m. UTC, <b>Albert Astals Cid</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">User documentation i mean, the .docbook file</pre>
 </blockquote>







</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I harmonised spelling of the player controllers using round brackets. I removed the terms "normal" and "hard" from their names as they are misleading. And I updated the documentation (a bit). Do you consider this issue fixed now or is there more to do regarding the documentation?</pre>
<br />




<p>- Alexander</p>


<br />
<p>On October 23rd, 2013, 2 a.m. UTC, Alexander Schuch wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for KDE Games.</div>
<div>By Alexander Schuch.</div>


<p style="color: grey;"><i>Updated Oct. 23, 2013, 2 a.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
konquest
</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Becai now plays quite well against Default Normal and Default Hard in nearly all situations - 1on1 and FFA. It is mainly tested in games with two other players, 22x22 board with 40 neutral planets with 0-2 neutral production.

It works nicely in cumulative games as well.

The only known "issue" is that it uses enemy kill percentage and defence fleet size for target selection even in blind games, where the player cannot see those values.</pre>
  </td>
 </tr>
</table>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>CMakeLists.txt <span style="color: grey">(b67b4ba)</span></li>

 <li>dialogs/newgamedlg.cc <span style="color: grey">(6f376ed)</span></li>

 <li>doc/index.docbook <span style="color: grey">(4e73b1d)</span></li>

 <li>players/ai/becai/becai.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>players/ai/becai/becai.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>players/ai/becai/becai_gui.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>players/ai/becai/becai_gui.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>players/ai/default/hard_gui.cpp <span style="color: grey">(e4e1568)</span></li>

 <li>players/ai/default/normal_gui.cpp <span style="color: grey">(41b2d31)</span></li>

 <li>players/ai/default/weak_gui.cpp <span style="color: grey">(feb76e9)</span></li>

 <li>players/ai/example/example_gui.cpp <span style="color: grey">(5e347ca)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/112747/diff/" style="margin-left: 3em;">View Diff</a></p>







  </td>
 </tr>
</table>








  </div>
 </body>
</html>