Skip to content

Commit 3050096

Browse files
committed
Address review comments
1 parent 00c6b34 commit 3050096

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

kadi/commands/commands_v2.py

+2
Original file line numberDiff line numberDiff line change
@@ -1022,6 +1022,8 @@ def update_cmd_events(
10221022
this environment variable is set, then events with a date after the value of
10231023
``CXOTIME_NOW`` are filtered out.
10241024
1025+
Finally, events are filtered by `event_filter` if it is provided.
1026+
10251027
Parameters
10261028
----------
10271029
scenario : str, None

kadi/commands/tests/test_filter_events.py

-3
Original file line numberDiff line numberDiff line change
@@ -166,9 +166,6 @@ def test_filter_scs107_events(
166166
# get_cmds() is working as expected - no CMD_EVT in the planned observations
167167
ok = cmds_planned["source"] == "CMD_EVT"
168168
assert np.all(cmds_planned[ok]["params"] == cmd_evt_params)
169-
170-
# As-run gets an extra observation because the manual obsid update at
171-
# 2025:001:12:48:34.040 breaks the observation into two.
172169
assert len(obss_as_run) == len_as_run
173170
assert len(obss_planned) == len_planned
174171

0 commit comments

Comments
 (0)