ExcitedStates / qfit-3.0

qFit: Automated and unbiased multi-conformer models from X-ray and EM maps.
MIT License
31 stars 11 forks source link

Split QP step into two in sample sidechain #389

Closed jessicaflowers closed 5 months ago

jessicaflowers commented 5 months ago

If too many conformers are generated during sidechain, split conformer array into two sections and score them separately

Pull Request Checklist


Description of the Change

If too many conformers are generated during sample_sidechain(), split conformer array into two sections and score them separately. Concatenate after scoring both sections. This allows us to sample >15000 conformers without overloading QP.

This is a temporary fix. Ideally, we want to reduce sampling as the time sampling is getting increasingly high for these residues. However, we believe we can do that more effectively with attention based sampling and so are leaving this patch in here for now.

Release Notes

Split QP during sample_sidechain() in qfit.py