-
Notifications
You must be signed in to change notification settings - Fork 10
Quick Fix to Bondi radii #25
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't workingdocumentationImprovements or additions to documentationImprovements or additions to documentationduplicateThis issue or pull request already existsThis issue or pull request already existsenhancementNew feature or requestNew feature or requestquestionFurther information is requestedFurther information is requestedwontfixThis will not be worked onThis will not be worked on
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingdocumentationImprovements or additions to documentationImprovements or additions to documentationduplicateThis issue or pull request already existsThis issue or pull request already existsenhancementNew feature or requestNew feature or requestquestionFurther information is requestedFurther information is requestedwontfixThis will not be worked onThis will not be worked on
Describe the bug
Current buried volume calculations include H-atoms by default. However, the Bondi radii for H-atoms (1.20 A) is not the radii used on popular buried volume tools such as SambVca. Instead, a radius of 1.10 A is used as "carbon bonded to hydrogen". Given the widespread use of SambVca as a tool for buried volume calculations, this discrepancy may cause confusion when different Vbur volumes are produced with SEQCROW.
On a separate note, I am not aware of any way to turn off H-atom inclusion, this would also be a good feature to add given they are frequently omitted in literature reported buried volumes.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The buried volume produced by SEQCROW should match that of SambVca