Skip to content

Incompatible options in dl_interface_design.py #101

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

Open
rszabla opened this issue Feb 7, 2025 · 0 comments
Open

Incompatible options in dl_interface_design.py #101

rszabla opened this issue Feb 7, 2025 · 0 comments

Comments

@rszabla
Copy link

rszabla commented Feb 7, 2025

I noticed that the options relax_cycles and seqs_per_struct are incompatible with each other. Basically, if seqs_per_struc is > 1, then relax_cycles must be set to 0. Otherwise the script throws an exception: "Struct with tag {tag} failed in 0 seconds with error: <class 'Exception'>"

Looking more closely, it looks like the function proteinmpnn_fastrelax was written with the assumption that there is only one seq per structure.
Is this an intentional behaviour? Would it be possible to change the function to allow multiple sequences per backbone?
I find that a seqs_per_struct of 2 is something of a sweet spot for my applications, so it would be nice to allow these structures to be relaxed with fast_relax after protein_MPNN does its thing.

Also, congrats on the Nobel prize! You guys definitely deserve it!

Thank you,

Robert Szabla
Western University
London, Canada

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

1 participant