Closed GoogleCodeExporter closed 9 years ago
Thanks for reporting the bug, together with the workaround.
Maybe Dave can have a look (I tentatively set owner to dcaplan)?
If nothing is happening for a week then I'll take a look.
Cheers,
Oliver
Original comment by orbeckst
on 28 Jul 2011 at 11:12
Btw, you could probably derive a custom HydrogenBondAnalysis class along the
lines of (untested):
from MDAnalysis.analysis.hbonds import HydrogenBondAnalysis
class LigandHydrogenBondAnalysis(HydrogenBondAnalysis):
donors = HydrogenBondAnalysis.donors + ('O3C', 'N2', 'O2', 'O6', 'O3', 'O4', 'O7', 'O8', 'O9', 'N5')
acceptors = HydrogenBondAnalysis.acceptors + ('O3C', 'OAA', 'O1H', 'O5', 'O6', 'O3', 'O4', 'O2', 'O7', 'O5N', 'O8', 'O9')
and the use the new class. Still not ideal but a little bit nicer than
monkey-patching.
Original comment by orbeckst
on 28 Jul 2011 at 11:21
Here's a patch that adds 2 arguments (donors, acceptors). Maybe Oliver can
merge it.
Original comment by dcaplan
on 29 Jul 2011 at 12:52
Attachments:
Added the patch and some smaller cosmetic changes in r826.
See also the updated Online Docs
http://mdanalysis.googlecode.com/svn/trunk/doc/html/documentation_pages/analysis
/hbonds.html
Please re-open the Issue if things are not working.
Thanks, Oliver
Original comment by orbeckst
on 29 Jul 2011 at 4:19
Er, the fix was in r829 ...
Original comment by orbeckst
on 29 Jul 2011 at 1:22
Original issue reported on code.google.com by
Brit.of....@googlemail.com
on 28 Jul 2011 at 1:47