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

[cm] Non-realtime wrapper #72

Open
wants to merge 3 commits into
base: cm--categorical_parameter
Choose a base branch
from

Conversation

christhetree
Copy link
Collaborator

@christhetree christhetree commented Feb 5, 2024

Defines a wrapper for non-realtime models making them compatible with the Neutone Gen plugin.
Most of the logic is in the constructor and the forward pass.

I wanted to allow the user to define NeutoneParameters in whatever order they like and all in one place rather than splitting up text and numerical parameters, which is why the constructor is a little bit more complicated than it would be otherwise.

One benefit is that we no longer need to worry about dynamic memory allocations which makes the code a bit cleaner in some cases.

@christhetree christhetree changed the base branch from main to cm--more_parameters February 5, 2024 18:30
@christhetree christhetree changed the base branch from cm--more_parameters to cm--string_parameter February 5, 2024 18:30
@christhetree christhetree force-pushed the cm--non_realtime_wrapper branch 3 times, most recently from f46b266 to 36045dc Compare February 5, 2024 22:14
@christhetree christhetree changed the base branch from cm--string_parameter to cm--categorical_parameter February 5, 2024 23:58
@christhetree christhetree force-pushed the cm--non_realtime_wrapper branch 2 times, most recently from 300d3ee to 7f48808 Compare February 6, 2024 13:05
@christhetree christhetree self-assigned this Feb 6, 2024
@christhetree christhetree force-pushed the cm--categorical_parameter branch 2 times, most recently from 70ef756 to 3609c96 Compare February 29, 2024 14:41
@christhetree christhetree force-pushed the cm--non_realtime_wrapper branch 2 times, most recently from e42ca44 to 7f8d2ac Compare March 4, 2024 15:48
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.

1 participant