-
Notifications
You must be signed in to change notification settings - Fork 2
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
Modify struct CNMF_results
#52
Comments
@degleris1 I think adding methods to the visualization function is a good way to address this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Several function take
CNMF_results
as an argument, including important visualization tools.However, if the user has a
W
andH
from some other algorithm, they must put in theCNMF_results
struct first and set a number of parameters tonothing
. I propose we either (a) overload these functions with their corresponding non-struct arguments or (b) change the last fields of the struct (l1_reg
,l2_reg
, ...) to a single fieldmodel_params
which contains a dictionary.The text was updated successfully, but these errors were encountered: