From 4d95f3304cea2f38393a95c0add00728c11e01db Mon Sep 17 00:00:00 2001 From: Brian Dubois Date: Tue, 16 May 2023 11:50:35 -0400 Subject: [PATCH] Remove unused timeSelectionCleared() action. --- tensorboard/webapp/metrics/actions/index.ts | 4 ---- tensorboard/webapp/metrics/store/metrics_reducers.ts | 6 ------ .../webapp/metrics/store/metrics_reducers_test.ts | 12 ------------ 3 files changed, 22 deletions(-) diff --git a/tensorboard/webapp/metrics/actions/index.ts b/tensorboard/webapp/metrics/actions/index.ts index 40f37a69d7..3cd3ebc823 100644 --- a/tensorboard/webapp/metrics/actions/index.ts +++ b/tensorboard/webapp/metrics/actions/index.ts @@ -220,10 +220,6 @@ export const cardViewBoxChanged = createAction( props<{cardId: CardId; userViewBox: Extent | null}>() ); -export const timeSelectionCleared = createAction( - '[Metrics] Linked Time Selection Cleared' -); - export const linkedTimeToggled = createAction( '[Metrics] Linked Time Enable Toggle', props<{ diff --git a/tensorboard/webapp/metrics/store/metrics_reducers.ts b/tensorboard/webapp/metrics/store/metrics_reducers.ts index 4a8489237e..133b43b9a7 100644 --- a/tensorboard/webapp/metrics/store/metrics_reducers.ts +++ b/tensorboard/webapp/metrics/store/metrics_reducers.ts @@ -1341,12 +1341,6 @@ const reducer = createReducer( cardStateMap: nextCardStateMap, }; }), - on(actions.timeSelectionCleared, (state) => { - return { - ...state, - linkedTimeSelection: null, - }; - }), on(actions.tableEditorTabChanged, (state, {tab}) => { return { ...state, diff --git a/tensorboard/webapp/metrics/store/metrics_reducers_test.ts b/tensorboard/webapp/metrics/store/metrics_reducers_test.ts index 0423e830d9..0aac2e5a32 100644 --- a/tensorboard/webapp/metrics/store/metrics_reducers_test.ts +++ b/tensorboard/webapp/metrics/store/metrics_reducers_test.ts @@ -3967,18 +3967,6 @@ describe('metrics reducers', () => { }); }); - describe('#timeSelectionCleared', () => { - it('clears linked time selection', () => { - const beforeState = buildMetricsState({ - linkedTimeSelection: {start: {step: 4}, end: {step: 4}}, - }); - - const nextState = reducers(beforeState, actions.timeSelectionCleared()); - - expect(nextState.linkedTimeSelection).toBeNull(); - }); - }); - describe('#rangeSelectionToggled', () => { it('toggles whether stepSelectorRange is enabled or not', () => { const state1 = buildMetricsState({