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

Probe 2.23 leads to slightly different results than 2.21 #11

Open
tillea opened this issue May 2, 2022 · 2 comments
Open

Probe 2.23 leads to slightly different results than 2.21 #11

tillea opened this issue May 2, 2022 · 2 comments

Comments

@tillea
Copy link

tillea commented May 2, 2022

Hi again
as I reported in issue #9 we now again have a change in the checksum of results between version 2.21 of probe (which was the last one we have packaged) and the latest version 2.23 I intend to package for Debian.
Is this change expected again? Would you be able to craft some test suite from your side we can simply run to prove expected behaviour?
Kind regards
Andreas.

@russell-taylor
Copy link
Collaborator

russell-taylor commented May 3, 2022

Andreas,

Thank you for updating the Debian packages and also for continuing to check to make sure the changes are desired.

In this case, I pulled some changes into the main branch so that the Richardson Lab can test them in MolProbity before we're sure that the results are better. I recommend that you maintain the previous version until that testing is complete. I'll let you know when that is.

The longer story is that we're developing a version of Probe that is embedded in the CCTBX Python library and our regression tests are turning up issues in the C code that should be improved. This one removes marking atoms in 5-membered Aromatic rings as Acceptors, which should prevent Hydrogen bonds to the rings from the side.

Thanks,
Russ

@tillea
Copy link
Author

tillea commented May 3, 2022 via email

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