Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new ax feature #109

Closed
wants to merge 8 commits into from
3 changes: 3 additions & 0 deletions gliderpy/plotting.py
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,9 @@ def plot_transect(
cbar = fig.colorbar(cs, orientation="vertical", extend="both")
cbar.ax.set_ylabel(var)
ax.set_ylabel("pressure")

ax.set_ylim(ax.get_ylim()[0], 0)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏽


return fig, ax


Binary file modified tests/baseline/test_plot_transect.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
44 changes: 26 additions & 18 deletions tests/test_plotting.py
Original file line number Diff line number Diff line change
@@ -1,36 +1,44 @@
"""Test transect."""

import cmcrameri
import pytest
import matplotlib as mpl
import matplotlib.pyplot as plt
from pathlib import Path
from gliderpy.plotting import plot_track
from gliderpy.plotting import plot_transect
from gliderpy.plotting import plot_track, plot_transect
from gliderpy.fetchers import GliderDataFetcher

root = Path(__file__).parent

@pytest.mark.mpl_image_compare(baseline_dir=root.joinpath("baseline/"))
def test_plot_track():
@pytest.fixture
def glider_data():
glider_grab = GliderDataFetcher()

glider_grab.fetcher.dataset_id = "whoi_406-20160902T1700"
df = glider_grab.to_pandas()
# Generate the plot
fig, ax = plot_track(df)
return df

@pytest.mark.mpl_image_compare(baseline_dir=root.joinpath("baseline/"))
def test_plot_track(glider_data):
# Generate the plot
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Generate the plot

fig, ax = plot_track(glider_data)
# Return the figure for pytest-mpl to compare
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Return the figure for pytest-mpl to compare

return fig



@pytest.mark.mpl_image_compare(baseline_dir=root.joinpath("baseline/"))
def test_plot_transect():
glider_grab = GliderDataFetcher()

glider_grab.fetcher.dataset_id = "whoi_406-20160902T1700"
df = glider_grab.to_pandas()
def test_plot_transect(glider_data):
# Generate the plot
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Generate the plot

fig, ax = plot_transect(df, 'temperature')
fig, ax = plot_transect(glider_data, 'temperature')
# Return the figure for pytest-mpl to compare
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Return the figure for pytest-mpl to compare

return fig

@pytest.mark.mpl_image_compare(baseline_dir=root.joinpath("baseline/"))
def test_plot_transect_multiple_figures(glider_data):
# Generate the plot with multiple figures
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Generate the plot with multiple figures

fig, (ax0, ax1) = plt.subplots(figsize=(15, 9), nrows=2, sharex=True, sharey=True)
glider_data.plot_transect(var="temperature", ax=ax0)
glider_data.plot_transect(var="salinity", ax=ax1, cmap=cmcrameri.cm.davos)
# Return the figure for pytest-mpl to compare
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Return the figure for pytest-mpl to compare

return fig

def test_plot_transect_size(glider_data):
# Generate the plot with a specific size
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Generate the plot with a specific size

fig, ax = plt.subplots(figsize=(15, 9))
glider_data.plot_transect(var="temperature", ax=ax)
assert fig.get_size_inches() == pytest.approx([15., 9.])