<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/D19393">View Revision</a></tr></table><br /><div><div><p>When Scheduler assigns a startup date to the job, it considers the altitude restriction and adds the altitude cutoff, then checks if the job has time to run.<br />
On the opposite, when Scheduler is executing the job, it only aborts it when the altitude restriction is crossed. The altitude cutoff is not considered.<br />
Because that's confusing to the end-user, I first provided the customization of that cutoff as an option. That's not extremely clearer that way, but at least a simpler test can be run by setting the cutoff to zero.</p>

<p>As I mentioned, the case that is of interest is the situation where the job has been executing for some time, and crosses the altitude restriction (or whatever restriction or operational issue that causes it to abort occurs).<br />
The runtime part of Scheduler marks it aborted, and the first block of evaluateJobs bypasses its re-evaluation. The job is then removed from the sorted list before evaluating the schedule.<br />
Yes, it ends up just after the last job that can still be run, but there's not much more the Scheduler can do while running.</p>

<p>So I'll be considering the JOB_ABORTED as a special case that must remain in the list in the original order.<br />
When only JOB_ABORTED jobs remain in the list, I'll reset them to</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/D19393">https://phabricator.kde.org/D19393</a></div></div><br /><div><strong>To: </strong>TallFurryMan<br /><strong>Cc: </strong>wreissenberger, kde-edu, narvaez, apol<br /></div>