Skip to content

Commit 4372402

Browse files
committed
docs: replace split matrices with single consolidated behavior matrix
1 parent b276d6d commit 4372402

1 file changed

Lines changed: 21 additions & 34 deletions

File tree

docs/data-tests/test-result-samples.mdx

Lines changed: 21 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -198,40 +198,27 @@ vars:
198198

199199
## Behavior matrix
200200

201-
The table below shows the resulting behavior for all combinations of the two feature flags.
202-
203-
### Only `enable_samples_on_show_sample_rows_tags: true`
204-
205-
| Level | Tags | Samples shown? |
206-
| --- | --- | --- |
207-
| model | none | No (hidden by default) |
208-
| model | `show_sample_rows` | Yes |
209-
| model | `pii` (only) | No (hidden by default, PII feature not active) |
210-
| test | `show_sample_rows` | Yes |
211-
| column | `show_sample_rows` | Yes |
212-
213-
### Only `disable_samples_on_pii_tags: true`
214-
215-
| Level | Tags | Samples shown? |
216-
| --- | --- | --- |
217-
| model | none | Yes |
218-
| model | `pii` | No |
219-
| test | `pii` | No |
220-
| column | `pii` | No |
221-
| column | `show_sample_rows` (no pii) | Yes (excluded from PII columns) |
222-
223-
### Both flags enabled
224-
225-
| Level | Tags | Samples shown? |
226-
| --- | --- | --- |
227-
| model | none | No (hidden by default) |
228-
| model | `show_sample_rows` | Yes |
229-
| model | `pii` | No (PII wins) |
230-
| model | `pii` + `show_sample_rows` | No (PII wins) |
231-
| test | `show_sample_rows` | Yes |
232-
| test | `pii` | No (PII wins) |
233-
| column | `show_sample_rows` (no pii) | Yes |
234-
| column | `pii` + `show_sample_rows` | No (PII wins) |
201+
| `enable_samples_on_show_sample_rows_tags` | `disable_samples_on_pii_tags` | Level | Tags | Samples shown? |
202+
| --- | --- | --- | --- | --- |
203+
| `true` | `false` | model | none | No (hidden by default) |
204+
| `true` | `false` | model | `show_sample_rows` | Yes |
205+
| `true` | `false` | model | `pii` | No (hidden by default, PII feature not active) |
206+
| `true` | `false` | test | `show_sample_rows` | Yes |
207+
| `true` | `false` | column | `show_sample_rows` | Yes |
208+
| `true` | `true` | model | none | No (hidden by default) |
209+
| `true` | `true` | model | `show_sample_rows` | Yes |
210+
| `true` | `true` | model | `pii` | No (PII wins) |
211+
| `true` | `true` | model | `pii` + `show_sample_rows` | No (PII wins) |
212+
| `true` | `true` | test | `show_sample_rows` | Yes |
213+
| `true` | `true` | test | `pii` | No (PII wins) |
214+
| `true` | `true` | column | `show_sample_rows` (no pii) | Yes |
215+
| `true` | `true` | column | `pii` + `show_sample_rows` | No (PII wins) |
216+
| `false` | `true` | model | none | Yes |
217+
| `false` | `true` | model | `pii` | No |
218+
| `false` | `true` | test | `pii` | No |
219+
| `false` | `true` | column | `pii` | No |
220+
| `false` | `true` | column | `show_sample_rows` (no pii) | Yes (excluded from PII columns) |
221+
| `false` | `false` | any | any | Yes (both features off) |
235222

236223
## Configuration precedence
237224

0 commit comments

Comments
 (0)