Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

[Android/IOS] Statistics always returning 0 for all properties #24

Closed
fayte0618 opened this issue Sep 11, 2018 · 14 comments
Closed

[Android/IOS] Statistics always returning 0 for all properties #24

fayte0618 opened this issue Sep 11, 2018 · 14 comments
Assignees
Labels
arm bug Something isn't working fixed

Comments

@fayte0618
Copy link

I'm currently using 2.1 version of your lib and mimicked your implementation in your test app using Config.enableStatisticsCallback. However, I noticed my StatisticsCallback isn't executed, and if I force get the statistics using Config.getLastReceivedStatistics I only receive all 0s. There's no error also. The command executes fine and exports the video. And I'm also using Config.enableLogCallback and it works fine. Why is this? Do I have to set this up or follow a series of steps before I use this?

Thanks for the help :)

@fayte0618
Copy link
Author

capture

@tanersener
Copy link
Owner

my StatisticsCallback isn't executed, and if I force get the statistics using Config.getLastReceivedStatistics I only receive all 0s. There's no error also.

You don't have to do anything else; it sounds like be bug, but I need to check it first.

Could you please provide the command you used if it is OK for you? I want to test it locally.

@fayte0618
Copy link
Author

-y -hide_banner -i [inputpath] -vf hue=s=0 -preset ultrafast -profile:v baseline -level 3.1 -crf 28 -c:a [outputpath]

if it helps im using the min-gpl version of your lib. But, when I switched to the others it still has the same bug.

@tanersener
Copy link
Owner

Tried your command and it worked for me, I received statistics callbacks.

To analyze it further, could you please post your command output? MobileFFmpeg basically extracts statistics from FFmpeg reports. Looking at command output, we can see whether FFmpeg generated reports or not?

@fayte0618
Copy link
Author

Here's the logs from the LogCallback
1
2
3

@tanersener
Copy link
Owner

Thanks. The following line belongs to an FFmpeg report printed.

logs

This is another report which has negative values inside. I'm not sure but they might be the reason of this issue. I'll make a few tests and post the results here.

two

@tanersener
Copy link
Owner

tanersener commented Sep 12, 2018

I tested statistics with negative values; they are not causing any trouble.

Is it possible to test your application with the following debug aar? This package includes extra debug logs for statistics, printed to LogCat. They may help to identify this issue.

mobile-ffmpeg-min-gpl-snapshot-2.2.aar

@fayte0618
Copy link
Author

Sorry for the late reply. I tried using your .aar but it gives me this error
capture

@tanersener
Copy link
Owner

No problem mate.

I managed to reproduce this issue on arm-v7a-neon. It looks like x86 variants are not affected from this, only arm ABIs I guess.

I'm working on it, I'll post an update after making some tests.

@tanersener tanersener added bug Something isn't working arm and removed needs-analysis labels Sep 14, 2018
@fayte0618
Copy link
Author

Thank you!

@tanersener
Copy link
Owner

Further tests show that armeabi-v7a and arm64-v8a are not affected from this issue, only armeabi-v7a-neon so far.

@tanersener
Copy link
Owner

Can you test this issue with v2.1.1?

@fayte0618
Copy link
Author

It works! thank you very much :))

@tanersener
Copy link
Owner

You're welcome. Thank you for helping me improve.

@tanersener tanersener changed the title Statistics always returning 0 for all properties [Android/IOS] Statistics always returning 0 for all properties Sep 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arm bug Something isn't working fixed
Projects
None yet
Development

No branches or pull requests

2 participants