Skip to content

Commit 8ca524b

Browse files
committed
Make h1x_gwss, _h1x_gwss, plot_h1x_gwss, plot_h1x_gwss_track, kwargs only. Make sample_query_options optional.
1 parent 7cd85d1 commit 8ca524b

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

malariagen_data/anoph/h1x.py

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ def __init__(
2626

2727
def _h1x_gwss(
2828
self,
29+
*,
2930
contig,
3031
analysis,
3132
window_size,
@@ -101,11 +102,12 @@ def _h1x_gwss(
101102
)
102103
def h1x_gwss(
103104
self,
105+
*,
104106
contig: base_params.contig,
105107
window_size: h12_params.window_size,
106108
cohort1_query: base_params.sample_query,
107109
cohort2_query: base_params.sample_query,
108-
sample_query_options: base_params.sample_query_options,
110+
sample_query_options: Optional[base_params.sample_query_options] = None,
109111
analysis: hap_params.analysis = base_params.DEFAULT,
110112
sample_sets: Optional[base_params.sample_sets] = None,
111113
cohort_size: Optional[base_params.cohort_size] = h12_params.cohort_size_default,
@@ -121,7 +123,7 @@ def h1x_gwss(
121123
) -> Tuple[np.ndarray, np.ndarray]:
122124
# Change this name if you ever change the behaviour of this function, to
123125
# invalidate any previously cached data.
124-
name = "h1x_gwss_v1"
126+
name = "h1x_gwss_v2"
125127

126128
params = dict(
127129
contig=contig,
@@ -161,10 +163,12 @@ def h1x_gwss(
161163
)
162164
def plot_h1x_gwss_track(
163165
self,
166+
*,
164167
contig: base_params.contig,
165168
window_size: h12_params.window_size,
166169
cohort1_query: base_params.cohort1_query,
167170
cohort2_query: base_params.cohort2_query,
171+
sample_query_options: Optional[base_params.sample_query_options] = None,
168172
analysis: hap_params.analysis = base_params.DEFAULT,
169173
sample_sets: Optional[base_params.sample_sets] = None,
170174
cohort_size: Optional[base_params.cohort_size] = h12_params.cohort_size_default,
@@ -195,6 +199,7 @@ def plot_h1x_gwss_track(
195199
max_cohort_size=max_cohort_size,
196200
cohort1_query=cohort1_query,
197201
cohort2_query=cohort2_query,
202+
sample_query_options=sample_query_options,
198203
sample_sets=sample_sets,
199204
random_seed=random_seed,
200205
chunks=chunks,
@@ -267,10 +272,12 @@ def plot_h1x_gwss_track(
267272
)
268273
def plot_h1x_gwss(
269274
self,
275+
*,
270276
contig: base_params.contig,
271277
window_size: h12_params.window_size,
272278
cohort1_query: base_params.cohort1_query,
273279
cohort2_query: base_params.cohort2_query,
280+
sample_query_options: Optional[base_params.sample_query_options] = None,
274281
analysis: hap_params.analysis = base_params.DEFAULT,
275282
sample_sets: Optional[base_params.sample_sets] = None,
276283
cohort_size: Optional[base_params.cohort_size] = h12_params.cohort_size_default,
@@ -298,6 +305,7 @@ def plot_h1x_gwss(
298305
window_size=window_size,
299306
cohort1_query=cohort1_query,
300307
cohort2_query=cohort2_query,
308+
sample_query_options=sample_query_options,
301309
sample_sets=sample_sets,
302310
cohort_size=cohort_size,
303311
min_cohort_size=min_cohort_size,

0 commit comments

Comments
 (0)