posebusters
posebusters copied to clipboard
Add timeout decorator to RMSD calculations to prevent occasional process deadlocks
- Adds a timeout decorator to the
_rmsdfunction to prevent occasional process deadlocks. - The timeout is set to 30 seconds by default.
Thank you for the suggestion. Three things would have to be addressed before we include a timeout in the RMSD module.
- All the tests must pass, including the "validation" check which checks that the pre-commit hooks ran.
- The
signalmodule in the standard Python library should be used instead of introducing a third party dependency. - It might change the results of the paper. I should add a test for this.
Out of personal interest, could you share which molecules cause these deadlocks? In my experience, highly symmetric molecules like Phytic acid cause (very) long run times because of the large number of possible bijections to calculate RMSDs for.
This PR was marked as stale because it has been open for 60 days with no activity.