D14942: Connect new-image event to Scheduler imaging time estimator.

Eric Dejouhanet noreply at phabricator.kde.org
Mon Sep 3 09:48:55 BST 2018


TallFurryMan added a comment.


  In D14942#317851 <https://phabricator.kde.org/D14942#317851>, @wreissenberger wrote:
  
  > Well, indeed, the feature name is not so self explanatory. But currently, the flag is not used inside of `updateCompletedJobsCount()`, right? As far as I understand, it is only used to estimate the duration. Looks like both methods could share a lot of code... OK, it looks like as there is room for improvement Capture.
  >
  > As a conclusion: I would recommend that we implement the following cache update algorithm:
  >
  > - determine updates for a  for `JOB_IDLE`and `JOB_EVALUATION` and collect these updates in `newFramesCount`
  > - merge these map into the old cache map in order to avoid overwriting with old cache values (not simply replacing the old cache)
  
  
  As a transitory and not satisfactory measure, I proposed D15230 <https://phabricator.kde.org/D15230>. Let's continue discussing there.

REPOSITORY
  R321 KStars

BRANCH
  improve__scheduler_image_update (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D14942

To: TallFurryMan, mutlaqja, wreissenberger
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180903/4b873bf0/attachment.html>


More information about the kde-edu mailing list