_Snaps.bind becomes _Snaps._bind

The call to bind in Step.isnap is unecessary since it is already
made is _Snaps.__getitem__. It hence makes sense to declare
_bind as an internal method of _Snaps.

Author: amorison

docs/sources/apiref/stagyydata.rst

@@ -4,4 +4,4 @@ stagyydata
 .. automodule:: stagpy.stagyydata
    :members:
    :private-members:
-   :exclude-members: bind, _as_view_item, _read_refstate, _pass
+   :exclude-members: _bind, _as_view_item, _read_refstate, _pass

stagpy/_step.py

@@ -443,7 +443,6 @@ def isnap(self):
             while (istep is None or istep < self.istep) and isnap < 99999:
                 isnap += 1
                 istep = self.sdat.snaps[isnap].istep
-                self.sdat.snaps.bind(isnap, istep)
                 # all intermediate istep could have their ._isnap to None
             if istep != self.istep:
                 self._isnap = None

stagpy/stagyydata.py

@@ -333,7 +333,7 @@ def __getitem__(self, isnap):
             else:
                 istep = None
             if istep is not None:
-                self.bind(isnap, istep)
+                self._bind(isnap, istep)
             else:
                 self._isteps[isnap] = None
         return self.sdat.steps[istep]
@@ -348,7 +348,7 @@ def __len__(self):
             if self.sdat.hdf5:
                 isnap = -1
                 for isnap, istep in stagyyparsers.read_time_h5(self.sdat.hdf5):
-                    self.bind(isnap, istep)
+                    self._bind(isnap, istep)
                 self._last = isnap
                 self._all_isteps_known = True
             if self._last < 0:
@@ -395,11 +395,9 @@ def at_time(self, time, after=False):
             igp -= 1
         return self[igp]
 
-    def bind(self, isnap, istep):
+    def _bind(self, isnap, istep):
         """Register the isnap / istep correspondence.
 
-        Users of :class:`StagyyData` should not use this method.
-
         Args:
             isnap (int): snapshot index.
             istep (int): time step index.