-
Notifications
You must be signed in to change notification settings - Fork 0
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
add number_metabomatching_permutations as an option #3
Comments
I don't think we should allow end users to change this. Why allowing them to change it and them warning them they should not? |
@BKhalili can you explain your motivation? |
Right now it is 99 in the code. I think you should allow for the change if a user wants to have a quick check. Like in our case that we want to check the code and results. |
There is an error when trying to run the docker and changing the number of permutations. |
Fixed, thanks! |
In metabomodules.py “number_metabomatching_permutations” variable is hardcoded please add it to parsers’ argument as it is impossible to change it for users using the docker. Also add a warning if it is set to less than 9999 that for the filtering to be reliable this variable should be set to 9999.
The text was updated successfully, but these errors were encountered: