-
Notifications
You must be signed in to change notification settings - Fork 25
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.
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
Opus file duration cannot be calculated #26
Comments
Please give more information. VorbisJava can't produce Opus file by itself, it can "transcode" them. I have used it quite a bit for that and did write some of the code for it. Can you use the opusinfo command from opus-tools on the bad file and give us the output of it? |
i have created OpusFile like that:
than writting opus packets 20ms length: and OpusInfoTool gives the following output: |
Thanks. The packets are not parsable it seems. Can you also try the opusinfo command from the native opus-tools from xiph? What is the output here? |
so the problem were in granule position, i'll try to make pr later |
Any updates on this? |
@msalikhov Thanks, that did the trick. |
The duration is rounded to seconds. It'd be great if it had precision down to milliseconds. |
I've merged the code from #31, with a few tweaks, so hopefully that'll work as standard. Thanks to @msalikhov for the help with this! @sepehr1014 How are you getting the duration that's rounded, so I can see if we can fix it / the info you need is already available elsewhere? |
@Gagravarr I was referring to |
I've just added a unit test How are you doing your reading of the duration where you are finding it truncated? |
cc @Sdghasemi |
Pull request which fixes a bug introduced in one of the commits related to this bug report: #33 |
@Gagravarr Android-Audio-Recorder is using Can you please merge mozbozz PR #33 and do a v0.9 release tag so that the bug may also be fixed in applications using |
So the created opus file duration cannot be determined by players. Probably because of granule position which is used for this by specs
The text was updated successfully, but these errors were encountered: