Repository navigation
Commit f93b9f6
Merge master (#1816) into integration/eval-rebuild
#1816 landed on master, so master now carries the hand-rolled per-evaluator
failure tail that #1846's _outcome.py refactor replaced here. Twelve source
files conflicted on exactly that: ours is the centralised form of theirs, so
ours wins everywhere. The structural guards in test_agentic_runner.py are what
confirm it -- they accept raise_agentic_failure() as one of the three ways a
kind can attach its records, and every multi-run kind still passes.
test_agentic_runner.py conflicted the usual way, ours a superset of master's.
Verified by AST rather than by reading: nothing defined on master is missing
here, and no definition is duplicated.
One defect the merge introduced silently, with no conflict to mark it:
test_trace_linker.py's _EVALUATE_FUNCS gained duplicate entries for
anomaly_detection, dashboard_summary and forecasting. Git auto-merged two
versions of a growing list by appending both. pytest refused to collect the
file over duplicate parametrize ids, which is the only reason it surfaced --
a list of tuples has no syntax error to trip over.
1771 passed, 2 skipped. The 16 fewer than before are precisely the 4 duplicate
entries times the 4 tests parametrized over that list.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>16 files changed
Lines changed: 66 additions & 47 deletions
File tree
- .openapi-generator/custom_templates
- gooddata-api-client
- packages
- gooddata-dbt
- gooddata-eval
- tests
- gooddata-fdw
- gooddata-flexconnect
- gooddata-flight-server
- gooddata-pandas
- tests/dataframe
- gooddata-pipelines
- gooddata-sdk
- tests/catalog
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
22 | | - | |
| 22 | + | |
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| |||
41 | 41 | | |
42 | 42 | | |
43 | 43 | | |
44 | | - | |
| 44 | + | |
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
| 28 | + | |
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | | - | |
| 40 | + | |
41 | 41 | | |
42 | 42 | | |
43 | 43 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
123 | 123 | | |
124 | 124 | | |
125 | 125 | | |
126 | | - | |
127 | | - | |
128 | | - | |
129 | | - | |
130 | 126 | | |
131 | 127 | | |
132 | 128 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
57 | | - | |
| 57 | + | |
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
| |||
77 | 77 | | |
78 | 78 | | |
79 | 79 | | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
79 | 79 | | |
80 | 80 | | |
81 | 81 | | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
86 | 86 | | |
87 | 87 | | |
88 | 88 | | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
0 commit comments