<table><tr><td style="">wreissenberger created this revision.<br />wreissenberger added reviewers: mutlaqja, TallFurryMan.<br />Herald added a project: KDE Edu.<br />Herald added a subscriber: kde-edu.<br />wreissenberger requested review of this revision.
</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><strong>REVISION SUMMARY</strong><div><p>Currently, the scheduler is a class with more than 7200 lines of code which makes it very tricky to maintain. This approach follows the idea to create a separate class ScheduleStrategy that holds the entire logic to select a certain schedule to be executed.</p>

<p>As a first step, the scoring calculation is extracted and shifted to ScheduleStrategy.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Well, this is the downside of this refactoring effort. Since the Scheduler is awfully complex and intertwined, it needs to be tested carefully.</p>

<p>At least from the intention, the execution of scheduler jobs should not be touched, only the correct calculations of scores. So please test all combinations that have effect to the score of a scheduler job.</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>AFFECTED FILES</strong><div><div>kstars/CMakeLists.txt<br />
kstars/ekos/scheduler/scheduler.cpp<br />
kstars/ekos/scheduler/scheduler.h<br />
kstars/ekos/scheduler/schedulerjob.cpp<br />
kstars/ekos/scheduler/schedulestrategy.cpp<br />
kstars/ekos/scheduler/schedulestrategy.h</div></div></div><br /><div><strong>To: </strong>wreissenberger, mutlaqja, TallFurryMan<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>