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

Why 5 diffusion samples? Possibility to set to 1 #109

Closed
NessimRaou opened this issue Nov 23, 2024 · 1 comment
Closed

Why 5 diffusion samples? Possibility to set to 1 #109

NessimRaou opened this issue Nov 23, 2024 · 1 comment
Labels
enhancement New feature or request question Further information is requested

Comments

@NessimRaou
Copy link

Thanks a lot for releasing the code. I have a question about it.
The paper mentions: "Using only one diffusion sample per model seed for the AF3 predictions rather than five (not illustrated) does not change the results significantly". Perhaps I missed how the decision to set 5 diffusion samples per seed was made but if it's true, I don't quite understand it.
Would it be possible to allow users to set it to 1? After modeling a very large protein (5900 residues, monomer), it substantially increases the computing time for similar output, so such a parameter may be useful in certain cases (especially for antibodies where more seeds improve the scores).

@Augustin-Zidek Augustin-Zidek added enhancement New feature or request question Further information is requested labels Nov 25, 2024
@Augustin-Zidek
Copy link
Collaborator

Hello, thank you for reporting this.

It is now possible to set the number of diffusion samples using the --num_diffusion_samples flag added in
1490230

I don't expect the performance impact to be very significant though on the model side (but the post-processing might be slow) -- would be curious to hear what the impact is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants