<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/D12485">View Revision</a></tr></table><br /><div><div><p>Well, I think I need an advice: scheduled jobs need a start time, but if multiple jobs start at the same time, the algorithm adding an offset to the start time will trigger.<br />
Before <a href="https://phabricator.kde.org/D12485" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12485</a>, jobs scheduled asap would not get a start time, so the algorithm adding the offset would manipulate invalid dates, and in the end be ineffective.<br />
So incidentally, by scheduling asap jobs to start "now", I properly have the algorithm adding the offset to valid dates, but this causes a regression because this offset now delays the startup of the jobs, which were contiguous before.<br />
However, this situation is probably due to the incorrect estimation of the duration of the job, which is used as offset. Well, actually, that offset is clamped to the lead time anyway, so jobs are separated by 5min minimum when they go through that part.<br />
So all in all, for this specific point, the previous behavior was somehow better than what the <a href="https://phabricator.kde.org/D12485" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12485</a> brings, so I'll probably revert if I don't find a more elegant solution, which might be to drop the algorithm adding offsets.</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/D12485">https://phabricator.kde.org/D12485</a></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja<br /><strong>Cc: </strong>KDE Edu, narvaez, apol<br /></div>