Bug 397970 - Yet another regression on Scheduler capture count
Summary: Yet another regression on Scheduler capture count
Status: CONFIRMED
Alias: None
Product: kstars
Classification: Applications
Component: general (show other bugs)
Version: git
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: TallFurryMan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-28 06:51 UTC by TallFurryMan
Modified: 2020-08-24 06:49 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Test case for the looping first job issue (968 bytes, application/x-gzip)
2018-08-29 11:33 UTC, TallFurryMan
Details
Test case for the frame count updating at each of sequence (943 bytes, application/x-gzip)
2018-08-29 11:34 UTC, TallFurryMan
Details

Note You need to log in before you can comment on or make changes to this bug.
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.