Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

EntropyTask processing videos when creating report slowing down report generation #1614

Closed
wladimirleite opened this issue Mar 31, 2023 · 0 comments · Fixed by #1616
Closed

EntropyTask processing videos when creating report slowing down report generation #1614

wladimirleite opened this issue Mar 31, 2023 · 0 comments · Fixed by #1616
Assignees
@wladimirleite
Labels
bug

Comments

@wladimirleite
Copy link
Member

This was reported to me by another user here.
In EntropyTask there is a check to skip items that have a thumbnail. When a report is being generated, this verification is not working properly.
I was able to reproduce and apparently fix the issue. I am running other tests, to check for undesired side effects, and expect to submit a fix soon.

This is not critical, but in the case that the issue was noticed, there were a lot of videos (about 800 GB) included in the report (with thumbsOnly option), and EntropyTask was consuming a relatively high percentage of the processing time.
@wladimirleite wladimirleite self-assigned this Mar 31, 2023
wladimirleite added a commit that referenced this issue Mar 31, 2023
@wladimirleite
'#1614: Fix log message when MPlayer is absent.
7029dc6
wladimirleite added a commit that referenced this issue Mar 31, 2023
@wladimirleite
'#1614: Check if VideoThumbTask is enabled after handling isIpedReport.
0639844
wladimirleite added a commit that referenced this issue Mar 31, 2023
@wladimirleite
'#1614: Set hasThumb as true, if it has view file and it is a report.
e9d4872
wladimirleite added a commit that referenced this issue Mar 31, 2023
@wladimirleite
'#1614: Fix last commit (set hasThumb was in the wrong place).
f899134
wladimirleite added a commit that referenced this issue Apr 1, 2023
@wladimirleite
'#1614: Minor fix in a comment.
f6c1c5e
@wladimirleite wladimirleite mentioned this issue Apr 1, 2023
Merged
lfcnassif added a commit that referenced this issue Apr 1, 2023
@lfcnassif
Merge pull request #1616 from sepinf-inc/#1614_FixReportVideoThumbs
b46aee8 
Set hasThumb when VideoThumbTask reuse view file (report processing) (#1614)
lfcnassif pushed a commit that referenced this issue Apr 13, 2023
@wladimirleite @lfcnassif
'#1614: Fix log message when MPlayer is absent.
53e0c24
lfcnassif pushed a commit that referenced this issue Apr 13, 2023
@wladimirleite @lfcnassif
'#1614: Check if VideoThumbTask is enabled after handling isIpedReport.
fca5f90
lfcnassif pushed a commit that referenced this issue Apr 13, 2023
@wladimirleite @lfcnassif
'#1614: Set hasThumb as true, if it has view file and it is a report.
e541114
lfcnassif pushed a commit that referenced this issue Apr 13, 2023
@wladimirleite @lfcnassif
'#1614: Fix last commit (set hasThumb was in the wrong place).
4e6e25e
lfcnassif pushed a commit that referenced this issue Apr 13, 2023
@wladimirleite @lfcnassif
'#1614: Minor fix in a comment.
e707ab7
@lfcnassif lfcnassif changed the title EntropyTask processing videos when generating a report EntropyTask processing videos when creating report slowing down report generation Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wladimirleite wladimirleite

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Set hasThumb when VideoThumbTask reuse view file (report processing) (#1614) sepinf-inc/IPED
1 participant
@wladimirleite