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

Increasing num_diffn_steps to 500 increased total_inter_chain_clashes #253

Open
Ruibin-Liu opened this issue Dec 17, 2024 · 3 comments
Open

Comments

@Ruibin-Liu
Copy link

I found that one chiral center of a ligand is inverted when I increased the num_diffn_timesteps from 100 to 500 and the total_inter_chain_clashes increased from 0 to 0.8. Is that expected? Is there a way to make sure the stereochemistry is always conserved in the difussion process?

@arogozhnikov
Copy link
Contributor

single observation doesn't carry enough information to draw conclusions; you need to run at least several examples (and compare several seeds).

My prior: I don't expect more steps to provide worse results (on average, not in particular case), but you're welcome to test this experimentally

@arogozhnikov arogozhnikov changed the title Chirality inverted for a ligand Increasing num_diffn_steps to 500 increased total_inter_chain_clashes Dec 17, 2024
@Ruibin-Liu
Copy link
Author

But I did observe that one chiral center is inverted, which is not desired for my case. I am not trying to assess the GENERAL'correctness' of Chai-1 but to know how to understand why that happened and how to avoid that.

@arogozhnikov
Copy link
Contributor

Simply reject samples that don't satisfy your requirements (e.g. chirality).

why that happened and how to avoid that.

random sampling and model not following guidance. To get any more detailed/useful explanation you need to check multiple relevant examples.

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

2 participants