<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="https://git.reviewboard.kde.org/r/116660/">https://git.reviewboard.kde.org/r/116660/</a>
     </td>
    </tr>
   </table>
   <br />










<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On March 8th, 2014, 4:52 p.m. UTC, <b>Frederik Gladhorn</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="https://git.reviewboard.kde.org/r/116660/diff/1/?file=252954#file252954line27" style="color: black; font-weight: bold; text-decoration: underline;">src/configure-practice/thresholdoptions.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

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



 
 

 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">27</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#include "practice/sessionmanager.h"</span></pre></td>
    <th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">27</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="c1"><span class="hl">//</span>#include "practice/sessionmanager<span class="hl">base</span>.h"</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;">If this include is no longer needed, why not just remove it.</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;">indeed</pre>
<br />




<p>- Inge</p>


<br />
<p>On March 8th, 2014, 4:44 p.m. UTC, Inge Wallin wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Edu and Amarvir Singh.</div>
<div>By Inge Wallin.</div>


<p style="color: grey;"><i>Updated March 8, 2014, 4:44 p.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
parley
</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;">This patch implement fixed sessions for Parley.  With that I mean that training is split up into sessions with a fixed number maximum number of words. The default value for the number is 20 but it could also be fewer. At most 5 of those can be new words, so it also makes it easier to learn new vocabulary by not allowing an unwieldy number of new words at a time. Before this patch a session was all the words that were due based on time since last practice and grade, which meant that a session could potentially consist of thousands of words.

Technically this patch works by splitting the old SessionManager into a base class (SessionManagerBase) and a specific class (SessionManagerContinuous) and then implementing a new class SessionManagerFixed.  The SessionManager was recently renamed into that name from the old TestEntryManager which didn't describe what it did at all.

This is the first milestone in the project to create a new session system.  Step 2 will be with adding some configuration and step 3 will be a new graphic look.

Note that in this version of the patch the numbers 20 (max size) and 5 (max new words) are replaced with 7 and 3 to make testing easier.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Tested with a special version of a word list I am working with.  I will attach it here.</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>src/settings/languagepropertiespage.cpp <span style="color: grey">(e324167)</span></li>

 <li>src/practice/writtenpracticevalidator.h <span style="color: grey">(d38190e)</span></li>

 <li>src/practice/writtenbackendmode.cpp <span style="color: grey">(4cb723e)</span></li>

 <li>src/practice/sessionmanagerfixed.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/writtenbackendmode.h <span style="color: grey">(dd8d3ee)</span></li>

 <li>src/practice/sessionmanagerfixed.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanagercontinuous.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanagercontinuous.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanagerbase.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanagerbase.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanager.cpp <span style="color: grey">(adea30b)</span></li>

 <li>TODO-SESSIONS <span style="color: grey">(PRE-CREATION)</span></li>

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

 <li>src/configure-practice/blockoptions.h <span style="color: grey">(1882994)</span></li>

 <li>src/configure-practice/thresholdoptions.cpp <span style="color: grey">(1fb55e1)</span></li>

 <li>src/parleymainwindow.h <span style="color: grey">(318312c)</span></li>

 <li>src/parleymainwindow.cpp <span style="color: grey">(1672647)</span></li>

 <li>src/practice/DESIGN <span style="color: grey">(99d4c0b)</span></li>

 <li>src/practice/comparisonbackendmode.h <span style="color: grey">(e5cb5d0)</span></li>

 <li>src/practice/comparisonbackendmode.cpp <span style="color: grey">(a3efac8)</span></li>

 <li>src/practice/conjugationbackendmode.h <span style="color: grey">(8cc59ee)</span></li>

 <li>src/practice/conjugationbackendmode.cpp <span style="color: grey">(fa365d9)</span></li>

 <li>src/practice/examplesentencebackendmode.h <span style="color: grey">(38941e1)</span></li>

 <li>src/practice/examplesentencebackendmode.cpp <span style="color: grey">(0cc812c)</span></li>

 <li>src/practice/genderbackendmode.h <span style="color: grey">(9a4a28a)</span></li>

 <li>src/practice/genderbackendmode.cpp <span style="color: grey">(b2cc7c1)</span></li>

 <li>src/practice/multiplechoicebackendmode.h <span style="color: grey">(a77265e)</span></li>

 <li>src/practice/multiplechoicebackendmode.cpp <span style="color: grey">(51f35a3)</span></li>

 <li>src/practice/practicemainwindow.h <span style="color: grey">(93fcd66)</span></li>

 <li>src/practice/practicemainwindow.cpp <span style="color: grey">(6fe7aa1)</span></li>

 <li>src/practice/practicestatemachine.h <span style="color: grey">(81cff91)</span></li>

 <li>src/practice/practicestatemachine.cpp <span style="color: grey">(16f86c1)</span></li>

 <li>src/practice/practicesummarycomponent.h <span style="color: grey">(41cc83b)</span></li>

 <li>src/practice/practicesummarycomponent.cpp <span style="color: grey">(bffb584)</span></li>

 <li>src/practice/sessionmanager.h <span style="color: grey">(64e70c9)</span></li>

</ul>

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



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>

<ul>

 <li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/03/08/f5213df1-cfe8-44cc-9d16-c2956bf39076__Thai-numbers-copy.kvtml">A small test file</a></li>

</ul>





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








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