ml-struct-bio/cryodrgn

Clean up ab initio argparse help

zhonge opened this issue · 1 comments

When I was cleaning up the cryodrgn2 docs, I noticed that some of the argument help strings in cryodrgn abinit_homo and cryodrgn abinit_het are missing the default values. Can you add those in @vineetbansal? Another minor improvement would be to make the help strings consistent for all the common input arguments across abinit_homo, abinit_het, train_vae, and train_nn.

I also noticed that cryodrgn abinit_homo doesn't have particle windowing turned on by default (unlike all the other tools)... We should update the default there asap.

zhonge commented

Merged @vineetbansal's changes in PR #264.

We'll release a minor version number to note the default behavior change for cryodrgn abinit_homo (no real space windowing by default).