Okay

Sent from my iPhone

On Apr 16, 2013, at 10:41 AM, kyleabeauchamp <notifications@github.com> wrote:

> In MDTraj/reporters/hdf5reporter.py:
> 
> > +        """
> > +        ######################################################################
> > +        # "/xyz" node, to store the coordinates
> > +        ######################################################################
> > +        if self._lossy:
> > +            xyz_atom = tables.Int16Atom()
> > +        else:
> > +            xyz_atom = tables.Float32Atom()
> > +
> > +        self._n_particles = simulation.system.getNumParticles()
> > +        try:
> > +            xyz_node = self._handle.getNode(where='/', name='xyz')
> > +            if xyz_node.shape[1] != self._n_particles:
> > +                raise ValueError('The number of atoms in the trajectory '
> > +                    'doesn\'t match the number of atoms in your system. '
> > +                    'Perhaps you should just save your trajectory to a '
> Here and directly below, maybe it make more sense to say "Perhaps you should save your trajectory to a different filename". My main beef is with the word "unique".
> 
> —
> Reply to this email directly or view it on GitHub.