Skip to content

Commit

Permalink
Merge pull request pvcaptest#110 from bt-/timeseries-filter-plot-bug
Browse files Browse the repository at this point in the history
Fixes bug with index of first timeseries filter scatter plot
  • Loading branch information
bt- committed May 6, 2024
2 parents 67c7414 + e715c40 commit 62110c5
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion src/captest/capdata.py
Original file line number Diff line number Diff line change
Expand Up @@ -1928,7 +1928,7 @@ def timeseries_filters(self):
)
plots.append(plt_no_filtering)

d1 = data.loc[self.removed[0]['index'], 'power']
d1 = data.loc[self.removed[0]['index'], ['power', 'Timestamp']]
plt_first_filter = hv.Scatter(
d1, ['Timestamp'], ['power'], label=self.removed[0]['name']
)
Expand Down
2 changes: 0 additions & 2 deletions tests/test_CapData.py
Original file line number Diff line number Diff line change
Expand Up @@ -2367,8 +2367,6 @@ def test_returns_overlay(self, meas):
})
meas.filter_irr(200, 900)
meas.filter_irr(400, 800)
# meas.data.index.name = 'Timestamp'
# meas.data_filtered.index.name = 'Timestamp'
overlay = meas.timeseries_filters()
assert 'index' not in meas.data.columns
assert 'index' not in meas.data_filtered.columns
Expand Down

0 comments on commit 62110c5

Please sign in to comment.