Commit 58563b2
authored
[O2B-1552] Move ad hoc filters inside runs overviews model to filtering model (#2106)
Notable changes for users:
- The monte carlo reproducable as not bad filters from the runs per .... overview pages has been changed to also be cleared when the 'reset all filters' button is pressed to be in line with all other filters.
Notable changes for developers:
**Filter restructuring**
- `ddflpFilter`, `dcsFilter`, and `epnFilter` have been moved to the `filteringModel`.
- `detectorsQc[_id][notBadFraction]` filters have been moved to a new model: `MultiCompositeFilterModel`.
- `gaq[notBadFraction]` now uses its own filter model with unique normalization logic: `GaqFilterModel`.
**New filter models**
- **`GaqFilterModel`**
- Dedicated model for `gaq[notBadFraction]`.
- Implements custom normalization logic.
- **`ToggleFilterModel`**
- Introduced for toggle-based filters.
- `mcReproducibleAsNotBad` has been converted to use this model.
- **`MultiCompositeFilterModel`**
- New flexible model:
- Can be instantiated with filters.
- Supports adding filters dynamically after instantiation.
- Computes normalized values based on:
- Insertion keys
- Normalized child filters
**Shared filter behavior**
- `mcReproducibleAsNotBad`:
- Converted into a `ToggleFilterModel`.
- Integrated into both:
- `GaqFilterModel`
- `MultiCompositeFilterModel`
- **Important:** both models must use the same instance for correct behavior.
**New components**
- `RadioButtonFilter`
- Replaces filters associated with the restructured filter models.
- `ToggleFilter` (component + model)
- Introduced alongside `ToggleFilterModel`.
**Renaming**
- `detectorsQc[_id][notBadFraction]` → `detectorsQcNotBadFraction[_id]`
- Reduces nesting within the `filterModel`.
**Implementation note**
- This approach avoids:
- Creating a custom wrapper around `NumericalComparisonFilterModel` solely for normalization, or
- Nesting `MultiCompositeFilterModel` within another `MultiCompositeFilterModel` to achieve automatic normalization.1 parent 9e26f06 commit 58563b2
34 files changed
Lines changed: 465 additions & 635 deletions
File tree
- lib
- domain/dtos/filters
- public
- components
- Filters
- LhcFillsFilter
- RunsFilter
- common
- filters
- common/form
- views
- LhcFills
- ActiveColumns
- Overview
- QcFlagTypes
- ActiveColumns
- Overview
- Runs
- ActiveColumns
- Overview
- RunPerDataPass
- RunPerPeriod
- RunsPerSimulationPass
- usecases/run
- test
- api
- lib/usecases/run
- public/runs
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
88 | 88 | | |
89 | 89 | | |
90 | 90 | | |
91 | | - | |
92 | | - | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
99 | 100 | | |
100 | 101 | | |
101 | 102 | | |
102 | | - | |
| 103 | + | |
103 | 104 | | |
104 | 105 | | |
105 | | - | |
| 106 | + | |
106 | 107 | | |
107 | 108 | | |
108 | 109 | | |
109 | 110 | | |
110 | | - | |
| 111 | + | |
111 | 112 | | |
112 | 113 | | |
113 | 114 | | |
| |||
117 | 118 | | |
118 | 119 | | |
119 | 120 | | |
120 | | - | |
| 121 | + | |
121 | 122 | | |
122 | 123 | | |
Lines changed: 0 additions & 76 deletions
This file was deleted.
Lines changed: 0 additions & 49 deletions
This file was deleted.
Lines changed: 89 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
Lines changed: 92 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
0 commit comments