Skip to content

Add stem export config menu for normalization#2330

Merged
m-m-adams merged 1 commit into
SynthstromAudible:communityfrom
seangoodvibes:Add-Config-Menu-for-Normalization
Jul 28, 2024
Merged

Add stem export config menu for normalization#2330
m-m-adams merged 1 commit into
SynthstromAudible:communityfrom
seangoodvibes:Add-Config-Menu-for-Normalization

Conversation

@seangoodvibes

@seangoodvibes seangoodvibes commented Jul 24, 2024

Copy link
Copy Markdown
Collaborator

This is a follow-up to the Stem Export menu PR. It adds a new configuration sub-menu with a single configuration setting (to start) to Enable or Disable the normalization of stems prior to export.

Refactored the recorder functions to accept "allowNormalization" as a flag from the caller to exclude normalization from the exported wav file.

@seangoodvibes seangoodvibes marked this pull request as draft July 24, 2024 22:46
@github-actions

github-actions Bot commented Jul 24, 2024

Copy link
Copy Markdown
Contributor

Test Results

75 tests  ±0   75 ✅ ±0   0s ⏱️ ±0s
13 suites ±0    0 💤 ±0 
13 files   ±0    0 ❌ ±0 

Results for commit 0f3e689. ± Comparison against base commit c367c04.

♻️ This comment has been updated with latest results.

@seangoodvibes seangoodvibes added this to the Release 1.2 milestone Jul 24, 2024
@seangoodvibes seangoodvibes force-pushed the Add-Config-Menu-for-Normalization branch 4 times, most recently from c4b0c53 to a641bae Compare July 27, 2024 18:30
@seangoodvibes seangoodvibes changed the title [DRAFT] Add stem export config menu for normalization Add stem export config menu for normalization Jul 27, 2024
@seangoodvibes seangoodvibes marked this pull request as ready for review July 27, 2024 18:32
@seangoodvibes seangoodvibes requested a review from m-m-adams July 27, 2024 18:35
Added configuration submenu with ability to Enable/Disable Normalization

Refactored the recorder functions to accept "allowNormalization" as a flag from the caller to exclude normalization from the exported wav file.
@seangoodvibes seangoodvibes force-pushed the Add-Config-Menu-for-Normalization branch from a641bae to 0f3e689 Compare July 27, 2024 18:36
@sichtbeton

Copy link
Copy Markdown
Contributor

this is great!

@m-m-adams m-m-adams added this pull request to the merge queue Jul 28, 2024
Merged via the queue into SynthstromAudible:community with commit 40c364d Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants