<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/D20001">View Revision</a></tr></table><br /><div><div><p>OK mostly, I'd rather have calculation pertaining to SchedulerJob relocated in SchedulerJob. "Strategy" would mean we have multiple ways of scheduling, which is not really the case today. We have "Restrictions" that schedules must comply with, and we have calculations leading to a score. I'm not a fan of that "score" value, but it pertains to the SchedulerJob, it should be the one owning the calculation. I think your previous work in moving stuff there was more interesting (but I can't find it anywhere...).<br />
Let me add an additional comment later on, I'm a bit busy now.</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/D20001">https://phabricator.kde.org/D20001</a></div></div><br /><div><strong>To: </strong>wreissenberger, mutlaqja, TallFurryMan<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>