Minimum mass for hydrogen guess in HydrogenBondAnalysis

package/CHANGELOG

@@ -81,6 +81,8 @@ Enhancements
     convert between a parmed.Structure and MDAnalysis Universe (PR #2404)
 
 Changes
+  * Added `min_mass` parameter to `guess_hydrogens` function in `HydrogenBondAnalysis`
+    set to 0.9 by default (Issue #2472)
   * Removes `save()` function from contacts, diffusionmap, hole, LinearDensity,
     and rms (Issue #1745).
   * Removes; `save_table()` from :class:`HydrogenBondAnalysis`,

package/MDAnalysis/analysis/hydrogenbonds/hbond_analysis.py

@@ -169,7 +169,7 @@
 """
 from __future__ import absolute_import, division
 
-import numpy as  np
+import numpy as np
 
 from .. import base
 from MDAnalysis.lib.distances import capped_distance, calc_angles
@@ -234,7 +234,12 @@ def __init__(self, universe, donors_sel=None, hydrogens_sel=None, acceptors_sel=
         self.d_h_a_angle = d_h_a_angle_cutoff
         self.update_selections = update_selections
 
-    def guess_hydrogens(self, selection='all', max_mass=1.1, min_charge=0.3):
+    def guess_hydrogens(self,
+                        selection='all',
+                        max_mass=1.1,
+                        min_charge=0.3,
+                        min_mass=0.9
+                        ):
         """Guesses which hydrogen atoms should be used in the analysis.
 
         Parameters
@@ -263,14 +268,20 @@ def guess_hydrogens(self, selection='all', max_mass=1.1, min_charge=0.3):
         Alternatively, this function may be used to quickly generate a :class:`str` of potential hydrogen atoms involved
         in hydrogen bonding. This str may then be modified before being used to set the attribute
         :attr:`hydrogens_sel`.
+
+        .. versionchanged: 1.0.0
+            Added `min_mass` parameter to specify minimum mass (Issue #2472)
         """
 
+        assert min_mass < max_mass
+
         ag = self.u.select_atoms(selection)
         hydrogens_ag = ag[
-            np.logical_and(
+            np.logical_and.reduce((
+                ag.masses > min_mass,
                 ag.masses < max_mass,
                 ag.charges > min_charge
-            )
+            ))
         ]
 
         hydrogens_list = np.unique(

testsuite/MDAnalysisTests/analysis/test_hydrogenbonds_analysis.py

@@ -139,7 +139,56 @@ def test_guess_donors(self, h):
         ref_donors = "(resname TIP3 and name OH2)"
         donors = h.guess_donors(selection='all', max_charge=-0.5)
         assert donors == ref_donors
+class TestHydrogenBondAnalysisTIP3P_GuessHyrdogens_NoTopology(object):
+    """
+    Guess the hydrogen atoms involved in hydrogen bonds using the mass and
+    partial charge of the atoms.
+    """
+
+    @staticmethod
+    @pytest.fixture(scope='class')
+    def universe():
+        return MDAnalysis.Universe(waterPSF, waterDCD)
+
+    kwargs = {
+        'donors_sel': None,
+        'hydrogens_sel': None,
+        'acceptors_sel': None,
+        'd_h_cutoff': 1.2,
+        'd_a_cutoff': 3.0,
+        'd_h_a_angle_cutoff': 120.0
+    }
+
+    @pytest.fixture(scope='class')
+    def h(self, universe):
+        h = HydrogenBondAnalysis(universe, **self.kwargs)
+        return h
+
+    def test_guess_hydrogens(self, h):
+
+        ref_hydrogens = "(resname TIP3 and name H1) or (resname TIP3 and name H2)"
+        hydrogens = h.guess_hydrogens(selection='all')
+        assert hydrogens == ref_hydrogens
+
+    def test_guess_hydrogens_min_mass(self, h):
+
+        hydrogens = h.guess_hydrogens(selection='all', min_mass=1.05)
+        assert hydrogens == ""
+
+    def test_guess_hydrogens_max_mass(self, h):
+
+        hydrogens = h.guess_hydrogens(selection='all', max_mass=0.95)
+        assert hydrogens == ""
+
+    def test_guess_hydrogens_min_charge(self, h):
+
+        hydrogens = h.guess_hydrogens(selection='all', min_charge=1.0)
+        assert hydrogens == ""
+
+    def test_guess_hydrogens_min_max_mass(self, h):
 
+        with pytest.raises(AssertionError):
+            hydrogens = h.guess_hydrogens(selection='all', min_mass=1.1, max_mass=0.9)
 
 class TestHydrogenBondAnalysisTIP3PStartStep(object):
     """Uses the same distance and cutoff hydrogen bond criteria as :class:`TestHydrogenBondAnalysisTIP3P` but starting