Skip to content

Commit b1f5c37

Browse files
Merge pull request #83 from AllenNeuralDynamics/change_multi_avg_signal_output
multi now returns nwbs not the df_trials
2 parents 02e31e0 + 52a7c6e commit b1f5c37

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

src/aind_dynamic_foraging_basic_analysis/metrics/trial_metrics.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -272,7 +272,6 @@ def get_average_signal_window_multi(
272272
pd.DataFrame
273273
Concatenated DataFrame of all trials with the new signal window column.
274274
"""
275-
all_trials_avg_signal = []
276275
for nwb in nwbs:
277276
df_trials = get_average_signal_window(
278277
nwb,
@@ -283,9 +282,8 @@ def get_average_signal_window_multi(
283282
censor=censor,
284283
output_col=output_col
285284
)
286-
cols_needed = ['trial', 'ses_idx', df_trials.columns[-1]]
287-
all_trials_avg_signal.append(df_trials[cols_needed])
288-
return pd.concat(all_trials_avg_signal, ignore_index=True)
285+
nwb.df_trials = df_trials
286+
return nwbs
289287

290288

291289
def get_average_signal_window(

0 commit comments

Comments
 (0)