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.

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

OggOpus records in 44.1 kHz but marks file as 48 kH #32

Open
6 of 7 tasks
RustoMCSpit opened this issue Feb 26, 2024 · 2 comments · May be fixed by #37
Open
6 of 7 tasks

OggOpus records in 44.1 kHz but marks file as 48 kH #32

RustoMCSpit opened this issue Feb 26, 2024 · 2 comments · May be fixed by #37
Labels
bug Something is not working

Comments

@RustoMCSpit
Copy link

Checklist

  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one feature request.
  • I have read and understood the contribution guidelines.
  • I optionally donated to support the Fossify mission.

Feature description

I made many recordings with Opus and 320 kbps, and realized they all sound high pitched (and a bit fast). Importing to Audacity and changing the sample rate to 44.1 fixes the problem.

Every audio player I've tried including the Simple Voice Recorder's playback of any recorded files sounds high-pitched.

It seems that the Ogg encoder the player uses encodes (or marks) the files as 48 kHz whereas they are being exported as 44.1 kHz.

The M4A setting does not have this problem, but I like Opus and very happy to see this program supports it.

Why do you want this feature?

As I liked opus

Additional information

This should be designed in a way that doesn't cause a dependancy issue for resampling in the future

@RustoMCSpit RustoMCSpit added feature request Issue is about a new feature in the app needs triage Issue is not yet ready for PR authors to take up labels Feb 26, 2024
@RustoMCSpit
Copy link
Author

This is a re-upload of a closed issue, no new info has been added.

@Aga-C Aga-C added bug Something is not working and removed feature request Issue is about a new feature in the app needs triage Issue is not yet ready for PR authors to take up labels Feb 26, 2024
Aga-C added a commit to Aga-C/Voice-Recorder that referenced this issue Feb 26, 2024
@Aga-C Aga-C linked a pull request Feb 26, 2024 that will close this issue
4 tasks
@Skirmisher
Copy link

Just noticed this bug personally, and realized the issue immediately (Opus doesn't encode at 44.1 kHz, but the input was recorded at that rate and not resampled to 48 kHz—see the Opus FAQ). Kind of disappointed that the Opus support exists with this kind of bug present, especially since the app is otherwise pretty nice...

(BTW, for the OP: there's no need to use a bit rate that high with Opus. 128 kbps is basically transparent for stereo sound, and I dunno if the average phone mic is even worth burning that much on. If you're only concerned with voice reproduction, you could go much lower if you wanted.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants