Skip to content

Commit a062eb8

Browse files
authored
Merge pull request #1127 from noir4201/fix/sample-indices-missing-methods
Fix: add sample_indices parameter to count_samples, plot_samples_interactive_map, plot_samples_bar, plot_sample_location_geo
2 parents 2fdabf4 + 509468c commit a062eb8

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

malariagen_data/anoph/sample_metadata.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -880,6 +880,7 @@ def count_samples(
880880
sample_sets: Optional[base_params.sample_sets] = None,
881881
sample_query: Optional[base_params.sample_query] = None,
882882
sample_query_options: Optional[base_params.sample_query_options] = None,
883+
sample_indices: Optional[base_params.sample_indices] = None,
883884
index: Union[str, Sequence[str]] = (
884885
"country",
885886
"admin1_iso",
@@ -894,6 +895,7 @@ def count_samples(
894895
sample_sets=sample_sets,
895896
sample_query=sample_query,
896897
sample_query_options=sample_query_options,
898+
sample_indices=sample_indices,
897899
)
898900

899901
# Create pivot table.
@@ -928,6 +930,7 @@ def plot_samples_interactive_map(
928930
sample_sets: Optional[base_params.sample_sets] = None,
929931
sample_query: Optional[base_params.sample_query] = None,
930932
sample_query_options: Optional[base_params.sample_query_options] = None,
933+
sample_indices: Optional[base_params.sample_indices] = None,
931934
basemap: Optional[map_params.basemap] = map_params.basemap_default,
932935
center: map_params.center = map_params.center_default,
933936
zoom: map_params.zoom = map_params.zoom_default,
@@ -947,6 +950,7 @@ def plot_samples_interactive_map(
947950
sample_sets=sample_sets,
948951
sample_query=sample_query,
949952
sample_query_options=sample_query_options,
953+
sample_indices=sample_indices,
950954
)
951955

952956
# Pivot taxa by locations.
@@ -1256,6 +1260,7 @@ def plot_samples_bar(
12561260
sample_sets: Optional[base_params.sample_sets] = None,
12571261
sample_query: Optional[base_params.sample_query] = None,
12581262
sample_query_options: Optional[base_params.sample_query_options] = None,
1263+
sample_indices: Optional[base_params.sample_indices] = None,
12591264
template: plotly_params.template = "plotly_white",
12601265
width: plotly_params.fig_width = 800,
12611266
height: plotly_params.fig_height = 600,
@@ -1268,6 +1273,7 @@ def plot_samples_bar(
12681273
sample_sets=sample_sets,
12691274
sample_query=sample_query,
12701275
sample_query_options=sample_query_options,
1276+
sample_indices=sample_indices,
12711277
)
12721278

12731279
# Special handling for plotting by year.
@@ -1768,6 +1774,7 @@ def plot_sample_location_geo(
17681774
sample_sets: Optional[base_params.sample_sets],
17691775
sample_query: Optional[base_params.sample_query] = None,
17701776
sample_query_options: Optional[base_params.sample_query_options] = None,
1777+
sample_indices: Optional[base_params.sample_indices] = None,
17711778
marker_size: plotly_params.marker_size = 10,
17721779
color: plotly_params.color = "admin1_name",
17731780
color_discrete_sequence: plotly_params.color_discrete_sequence = px.colors.qualitative.Prism,
@@ -1786,6 +1793,7 @@ def plot_sample_location_geo(
17861793
sample_sets=sample_sets,
17871794
sample_query=sample_query,
17881795
sample_query_options=sample_query_options,
1796+
sample_indices=sample_indices,
17891797
)
17901798

17911799
# Set the location columns to use from the sample metadata.

0 commit comments

Comments
 (0)