Skip to content

Migrate Melgan Spectrogram Inversion to Keras3 - #2347

Open
maitry63 wants to merge 3 commits into
keras-team:masterfrom
maitry63:melgan_inversion_keras3
Open

Migrate Melgan Spectrogram Inversion to Keras3#2347
maitry63 wants to merge 3 commits into
keras-team:masterfrom
maitry63:melgan_inversion_keras3

Conversation

@maitry63

Copy link
Copy Markdown
Collaborator

This PR Migrate the Melgan Spectrogram Inversion Tutorial to Keras 3.

Notebook - Colab

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request migrates the MelGAN spectrogram inversion example to Keras 3, replacing TensorFlow-specific operations and dependencies with Keras 3 ops and a custom WeightNormalization layer. Feedback highlights several issues: the WeightNormalization layer is missing dilation_rate support and a get_config method; the generator's upsampling blocks were incorrectly changed to Conv1D; redundant code was found in the MelSpec layer; and the discriminator's pooling size was altered, potentially impacting model architecture.

Comment thread examples/audio/melgan_spectrogram_inversion.py Outdated
Comment thread examples/audio/melgan_spectrogram_inversion.py Outdated
Comment thread examples/audio/melgan_spectrogram_inversion.py
Comment thread examples/audio/melgan_spectrogram_inversion.py
Comment thread examples/audio/melgan_spectrogram_inversion.py Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants