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

Add audio normalization function #609

Merged
1 commit merged into from
Jul 17, 2017
Merged

Add audio normalization function #609

1 commit merged into from
Jul 17, 2017

Conversation

dspinellis
Copy link
Contributor

Issue: #32

@coveralls
Copy link

coveralls commented Jul 4, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.006%) to 54.076% when pulling 6f7944a on dspinellis:audio_normalize into 6ba12d0 on Zulko:master.

@coveralls
Copy link

coveralls commented Jul 4, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.6%) to 54.665% when pulling a587e70 on dspinellis:audio_normalize into 6ba12d0 on Zulko:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 54.665% when pulling 582ec7f on dspinellis:audio_normalize into 6ba12d0 on Zulko:master.

1 similar comment
@coveralls
Copy link

coveralls commented Jul 4, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.6%) to 54.665% when pulling 582ec7f on dspinellis:audio_normalize into 6ba12d0 on Zulko:master.

kencochrane
kencochrane approved these changes Jul 8, 2017
View reviewed changes
Copy link
Contributor

@kencochrane kencochrane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, thanks LGTM

@tburrows13 tburrows13 added the feature New addition to the API i.e. a new class, method or parameter. label Jul 8, 2017
@dspinellis
Copy link
Contributor Author

Thank you! Is there anything else you want from me in order to merge it?

@kencochrane
Copy link
Contributor

I'm not a maintainer, so I'm not able to merge any PRs, but maybe @Gloin1313 or @mbeacom can take a look when they have a free minute, and if they are happy, hopefully they can merge.

@ghost ghost merged commit 9664b38 into Zulko:master Jul 17, 2017
@mondeja mondeja mentioned this pull request Jan 21, 2021
Open
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kencochrane kencochrane kencochrane approved these changes

Assignees
No one assigned
Labels
feature New addition to the API i.e. a new class, method or parameter.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dspinellis @coveralls @kencochrane @tburrows13