<table><tr><td style="">TallFurryMan added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D14684">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D14684#305563" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D14684#305563</a>, <a href="https://phabricator.kde.org/p/mutlaqja/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mutlaqja</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><ol class="remarkup-list">
<li class="remarkup-list-item">The capture bar is showing 100% as soon as it starts then goes to normal. Bar should advanced _after_ an image is captured, not before/during since it's confusing to see 100% and still capturing last image.</li>
<li class="remarkup-list-item">Capture logs says 2018-08-09T09:43:11 Job requires 5-second Red images, has 5/0 frames captured and will be processed. which probably needs to be reversed 0/5</li>
<li class="remarkup-list-item">Capturing 5-second Red image... --> not sure about format here. I think the dash needs to be dropped. Capturing 5 seconds image. Maybe need to use i18np for singular/plural case but it's double value so that might be tricky.</li>
</ol></div>
</blockquote>



<ol class="remarkup-list">
<li class="remarkup-list-item">I can't reproduce that in the latest diff if we're talking about the Capture module progress bar. There is, however, a concurrency when using 1-second captures which causes the progress to remain at 100% while processing single-capture sequence jobs, but that's very elusive. With greater exposures I don't see the issue. If actually it shows up on lower-end computers I'll have a look, please tell me.</li>
<li class="remarkup-list-item">Fixed</li>
<li class="remarkup-list-item">I'll leave it like this, please tell me if you think otherwise. That construct has the advantage to bypass the management code for plurals :)</li>
</ol>

<blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D14684#306156" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D14684#306156</a>, <a href="https://phabricator.kde.org/p/wreissenberger/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@wreissenberger</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Not sure whether the problem still exists after your changes, but I found a situation where the scheduler loops endlessly.<br />
 If you have two jobs with the same capture file signature (path + filter etc.) and the <strong>second</strong> has <strong>less</strong> repeats than the first, the first job will repeat endlessly.<br />
 The problem is that Scheduler::updateCompletedJobsCount() overwrites the map entry such that a later evaluation thinks that it has less captures than requested.</p></div>
</blockquote>

<p>Aha, let's have a look into this, thanks. I did actually check for this situation in a more general manner, that is, code should not presume of the order in which the scheduler jobs are sorted in the list, but I should have built a test vector.<br />
-Eric</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14684">https://phabricator.kde.org/D14684</a></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja, wreissenberger<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>