Bug 397970

Summary: Yet another regression on Scheduler capture count
Product: [Applications] kstars Reporter: TallFurryMan <eric.dejouhanet>
Component: generalAssignee: TallFurryMan <eric.dejouhanet>
Status: CONFIRMED ---    
Severity: normal    
Priority: NOR    
Version: git   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Test case for the looping first job issue
Test case for the frame count updating at each of sequence

Description TallFurryMan 2018-08-28 06:51:36 UTC
A last (?) regression on the Scheduler capture count.

- Use a sequence file with 5-second 3xR 3xG 3xR.
- Execute Scheduler on this sequence, wait for it to finish.
- Remove one Red capture.
- Refresh Scheduler plan, missing capture is spotted.
- Execute Scheduler plan, Frame Map summary is wrong and 3 Red captures are processed.
Comment 1 TallFurryMan 2018-08-29 11:33:24 UTC
Two possible extensions to this issue from @wreissenberger:

"The setup is the following: create a schedule with two jobs for the same target with the same sequence job (say 1xLum for example). The first job starts and captures the first frame.

And here is where the problem occurs: now calling Scheduler::updateCompletedJobsCount() detects the one frame, but when evaluating the second job (which did not run yet), the value in the captured frames map is overwritten. This leads to an endless loop of the first job."

"I observed that the counting update happens only after a single sequence is finished and not after each capture."
Comment 2 TallFurryMan 2018-08-29 11:33:45 UTC
Created attachment 114674 [details]
Test case for the looping first job issue
Comment 3 TallFurryMan 2018-08-29 11:34:09 UTC
Created attachment 114675 [details]
Test case for the frame count updating at each of sequence
Comment 4 TallFurryMan 2018-09-03 08:51:54 UTC
For FIRST EXTENSION of this issue: "This leads to an endless loop of the first job".
Proposed hotfix at https://phabricator.kde.org/D15230.
Comment 5 TallFurryMan 2020-08-24 06:49:48 UTC
Will verify this issue with Capture tests soon.