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

[FEATURE] Pre-convolution for LSTM #289

Open
sdatkinson opened this issue Jul 3, 2023 · 2 comments
Open

[FEATURE] Pre-convolution for LSTM #289

sdatkinson opened this issue Jul 3, 2023 · 2 comments
Labels
architectures New modeling architectures enhancement New feature or request priority:low Low-priority issues

Comments

@sdatkinson
Copy link
Owner

Option to incorporate a convolutional layer in front of LSTM models. Helps w/ sensitivity to delay, and generally improves training.

@sdatkinson sdatkinson added enhancement New feature or request unread This issue is new and hasn't been seen by the maintainers yet priority:low Low-priority issues and removed unread This issue is new and hasn't been seen by the maintainers yet labels Jul 3, 2023
@38github
Copy link

38github commented Nov 7, 2023

As an LSTM fan I have to ask - how is this going?

@38github
Copy link

38github commented Nov 7, 2023

My (subjective and objective) findings between WaveNet and LSTM is:

WaveNet - Pros

  • Achieves good frequency response even in extreme cases like EQs
  • Manages to model some ambience
  • Easy to train

WaveNet - Cons

  • Unbearable higher frequencies that becomes more prominent the more distortion the model has. Sounds like aliasing and in the end all high gain amplifiers/pedals/etc sound about the same.
  • Post-ringing
  • CPU usage

LSTM - Pros

  • Distortion sounds very good and no audible aliasing like sounds.
  • No post-ringing
  • Low CPU

LSTM - Cons

  • Does not achieve very good frequency response with extreme EQing like WaveNet does
  • Loses some higher frequencies at lower quality BUT is solved by increasing mostly hidden size but also increases CPU usage
  • Hard time handling ambience.
  • Very finicky with training parameters

@sdatkinson sdatkinson added the architectures New modeling architectures label Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
architectures New modeling architectures enhancement New feature or request priority:low Low-priority issues
Projects
None yet
Development

No branches or pull requests

2 participants