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

box size #353

Closed
dhiraj82 opened this issue Mar 6, 2024 · 2 comments
Closed

box size #353

dhiraj82 opened this issue Mar 6, 2024 · 2 comments

Comments

@dhiraj82
Copy link

dhiraj82 commented Mar 6, 2024

Hi
It's not really an issue but I have a box size of 324 which is not divisible by 8. even if I am downsampling it by 2 or 3 or 4, it will not be divisible by 8. How can I get around this problem? is it ok to downsample it to 128? then how am i going to train cryodrgn without downsampling?

Thank you
Dhiraj

@ryanfeathers
Copy link
Contributor

Hi Dhiraj,

You can definitely downsample by other factors! When downsampling to 128, multiply your pixel size by 324/128 and provide this value for cryodrgn analyze --Apix

Best,

Ryan

@michal-g
Copy link
Collaborator

michal-g commented May 6, 2024

As of v3.3.1 we are removing the error that is thrown when train_vae and train_nn inputs are not divisible by 8, instead throwing a warning that AMP training is not optimized when not using --no-amp (divisibility by 8 is only an issue when using AMP).

But yes, it is ok to downsample to 128 as Ryan answered above — you may not be able to run reconstruction as fast as otherwise, but you should't encounter any other issues because of this.

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

No branches or pull requests

3 participants