Skip to content

Commit

Permalink
Merge pull request #37 from theavey/ParaTemp-35
Browse files Browse the repository at this point in the history
#35 GromacsWrapper ≥ 0.7 supports Python 3.4+
  • Loading branch information
theavey authored Nov 27, 2018
2 parents 91b94f7 + 9a0b061 commit 5cafd78
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 21 deletions.
8 changes: 0 additions & 8 deletions paratemp/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,6 @@

from __future__ import absolute_import

import sys

from . import para_temp_setup
from .tools import copy_no_overwrite, cd, get_temperatures
from . import coordinate_analysis
Expand All @@ -37,12 +35,6 @@

del absolute_import

if sys.version_info.major == 2:
# These (at this point) require python 2 because of gromacs (gromacswrapper)
from . import energy_histo
from . import energy_bin_analysis
del sys


__version__ = get_versions()['version']
del get_versions
Expand Down
8 changes: 4 additions & 4 deletions paratemp/coordinate_analysis.py
Original file line number Diff line number Diff line change
Expand Up @@ -410,7 +410,7 @@ def fes_1d(self, data, bins=None, temp=None,
:param bins: Default: None. The bins argument to be passed to
np.histogram
:param str xlabel: Default: 'distance / $\mathrm{\AA}$'. The label for
:param str xlabel: Default: 'distance / $\\mathrm{\\AA}$'. The label for
the x axis.
:type ax: matplotlib.axes.Axes
Expand Down Expand Up @@ -935,7 +935,7 @@ def plot_ox_dists(self, save=False, save_format='png',
**kwargs)
ax.legend()
ax.set_xlabel('time / ps')
ax.set_ylabel('distance / $\mathrm{\AA}$')
ax.set_ylabel(r'distance / $\mathrm{\AA}$')
if save:
fig.savefig(save_base_name + save_format)
if display:
Expand Down Expand Up @@ -1194,7 +1194,7 @@ def make_plot_taddol_ox_dists(data, save=False, save_format='pdf',
axes.plot(data[:, 0], data[:, 3], label='O(r)-Cy')
axes.legend()
axes.set_xlabel('time / ps')
axes.set_ylabel('distance / $\mathrm{\AA}$')
axes.set_ylabel(r'distance / $\mathrm{\AA}$')
if save:
fig.savefig(save_base_name+save_format)
if display:
Expand Down Expand Up @@ -1279,7 +1279,7 @@ def make_taddol_pi_dist_array(dists, save=False, save_format='pdf',
"""Plot array of pi distances in TADDOL trajectory"""
fig = plot_dist_array(dists)
[ax.get_xaxis().set_ticks([]) for ax in fig.axes]
fig.text(0.05, 0.585, 'distance / $\mathrm{\AA}$', ha='center',
fig.text(0.05, 0.585, r'distance / $\mathrm{\AA}$', ha='center',
rotation='vertical')
fig.text(0.513, 0.08, 'time', ha='center')
if save:
Expand Down
7 changes: 1 addition & 6 deletions paratemp/energy_histo.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,7 @@
import pandas as pd

from .tools import all_elements_same

__version__ = '0.0.2'

# todo define a set of run data as a class
# possibly as a class of mdanalysis universes, but not sure how to work with
# replicas vs. walkers there.
from . import __version__

# TODO update docstrings to PEP specs
# (one brief description line, blank line, longer description/guidelines)
Expand Down
2 changes: 1 addition & 1 deletion paratemp/plotting.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ def fes_1d(x, temp, ax=None, bins=None,
:param bins: Default: None. The bins argument to be passed to
np.histogram
:param str xlabel: Default: 'distance / $\mathrm{\AA}$'. The label for
:param str xlabel: Default: 'distance / $\\mathrm{\\AA}$'. The label for
the x axis.
:type ax: matplotlib.axes.Axes
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ numpy
matplotlib
panedr
py
gromacswrapper
gromacswrapper>=0.7
tables
typing
scipy
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
'numpy',
'matplotlib',
'panedr',
'gromacswrapper',
'gromacswrapper>=0.7',
'tables',
'typing',
'scipy',
Expand Down

0 comments on commit 5cafd78

Please sign in to comment.