Skip to content

feat: audio input gain control#800

Merged
TimPietrusky merged 10 commits intonextfrom
feat/727/gain
Jan 3, 2023
Merged

feat: audio input gain control#800
TimPietrusky merged 10 commits intonextfrom
feat/727/gain

Conversation

@TimPietrusky
Copy link
Copy Markdown
Member

See #727

@TimPietrusky TimPietrusky requested a review from 2xAA December 13, 2022 22:06
@TimPietrusky TimPietrusky marked this pull request as draft December 13, 2022 22:12
@TimPietrusky
Copy link
Copy Markdown
Member Author

  • Set the gain via "Input Device Config"
  • Use the gainNode in meyda

@2xAA 2xAA marked this pull request as ready for review December 14, 2022 23:03
Comment thread src/components/InputDeviceConfig/AudioVideo.vue
Comment thread src/components/InputDeviceConfig/AudioVideo.vue
Comment thread src/components/InputDeviceConfig/AudioVideo.vue
switchingAudio: false,
switchingVideo: false
switchingVideo: false,
gainRangeValue: 1
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Using a value on data so we can keep track of what the value is doing.
Vue wasn't aware of the value before this.

We also update the value when the component is created below in the created function.

@2xAA
Copy link
Copy Markdown
Member

2xAA commented Dec 14, 2022

@TimPietrusky If you're happy with this PR, we can merge!

@2xAA 2xAA changed the title feat/727/gain feat: audio input gain control Dec 14, 2022
@2xAA 2xAA linked an issue Dec 14, 2022 that may be closed by this pull request
@TimPietrusky
Copy link
Copy Markdown
Member Author

TimPietrusky commented Dec 17, 2022

I will not merge this yet, as it's "hard" to get back to 1 or any specific value, as we can't use a right click on the slider as with the other slider we have. I will use the branch for my gig, but we should refactor this so that it's also possible to set the value via input field.

Also the tooltip of the range-slider and the value shown on the right are not the same in some cases, which is misleading.

@2xAA
Copy link
Copy Markdown
Member

2xAA commented Dec 21, 2022

@TimPietrusky Ready to review again. This should have addressed your comments

@TimPietrusky TimPietrusky merged commit 07ac0a0 into next Jan 3, 2023
@TimPietrusky TimPietrusky deleted the feat/727/gain branch January 3, 2023 21:53
github-actions Bot pushed a commit that referenced this pull request Mar 1, 2023
# [3.26.0](3.25.5...3.26.0) (2023-03-01)

### Bug Fixes

* **checkbox:** add boolean emission prop ([#792](#792)) ([e62f47a](e62f47a))
* **color-picker:** dismiss color picker by clicking outside color picker ([#793](#793)) ([015b84a](015b84a)), closes [#461](#461)
* **plugins:** emit boolean from checkbox ([#795](#795)) ([c13f54a](c13f54a))
* **styling:** fix unwanted top range control alignment in module inspector ([#806](#806)) ([0082450](0082450))
* **styling:** update tab styles to match design. fix tab dropdown stacking ([#815](#815)) ([a280099](a280099))
* **swap:** properly set default swap structure when finished swapping ([#766](#766)) ([144453c](144453c)), closes [#765](#765)

### Features

* add animated loader, remove when loaded to prevent redraws ([#771](#771)) ([57f3658](57f3658))
* add automation of browserslist update ([#787](#787)) ([16f33a6](16f33a6)), closes [#691](#691)
* audio input gain control ([#800](#800)) ([07ac0a0](07ac0a0))
* **gallery-item:** implement better gallery item scaling ([#791](#791)) ([9146c1e](9146c1e))
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 1, 2023

🎉 This PR is included in version 3.26.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Increase audio input gain directly in meyda

2 participants