-
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
Fix bugs and recent developments in MDSPACE & MDTOMO #189
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job and clean code!
PDB_SOURCE_OBJECT = 1 | ||
PDB_SOURCE_TRAJECT = 2 | ||
class FlexProtCA2AA(ProtAnalysis3D): | ||
""" Protocol to convert at set of carbon-alpha PDBs to all-atom PDBs using a reference all-atom PDB. """ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello Remi
Out of curiosity, how are you using this function?
form.addParam('reducedSpace', params.PathParam, label="Provide a txt file of the existing reduced space", condition="loadReducedSpace", | ||
help="Cloud of point of N dimension constituing a reduced space " | ||
" obtained by any reduction method. The file is a txt file.", expertLevel=params.LEVEL_ADVANCED) | ||
|
||
form.addParam('method', params.EnumParam, label="Reduction method", default=REDUCE_METHOD_PCA, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing "Dimensionality" before "Reduction method"
group.addParam('freeEnergySize', IntParam, default=100, | ||
label='Sampling size' ) | ||
group.addParam('freeEnergySize', IntParam, default=50, | ||
label='Resolution (pix)' ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolution is already an abused term, maybe we replace it by "window size (pix)"? otherwise, add some description in help
Some fixes and recent developments