Skip to content

Commit 62b339c

Browse files
committed
spec R51: trim default-visible columns to match spec list
R51 lists 11 default-visible columns (clonotype key, dev/integrity risks, dev score, surfacedMotifCount, exposed/broken cys counts, PSH/PPC/PNC flags, mode-specific flag). Two columns were visible by default but not in that list: - `mode` ("Mode (R7)") flips to optional. The mode prefix is also surfaced in the R55 subtitle, so users see it without taking up a table column. - `totalCdrLengthFlag` flips to optional. The raw `totalCdrLength` metric is explicitly listed under "Hidden behind show all"; its flag follows the same hidden-by-default treatment. Inlined the flag column spec (instead of `flagColumnSpec(...)`) since this is the only flag with a non-default visibility. Other 5 deviations flagged during the audit were left alone: - R25 r12 < 7.5 Å vs ≤ 7.5 Å is measure-zero on continuous distances. - R39 VHH compactness band edge 1.56 vs 1.57 follows TNP source (spec calls it "verbatim"); spec table boundary overlap at 1.57. - R23 buried-extra-cys "proportionally less via R20 logistic" contradicts the R41 hard step (`exposedExtraCysCount × 8.0`); code matches the explicit R41 formula. - R22 cys hit "carries region" not implemented; per-cys PFrame is out-of-scope per the refreshed spec ("Out of scope: per-residue liability drill-down").
1 parent f64bae2 commit 62b339c

1 file changed

Lines changed: 16 additions & 4 deletions

File tree

workflow/src/specs.lib.tengo

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ scoresColumnSpecs := {
107107
name: "pl7.app/liabilities/mode",
108108
annotations: {
109109
"pl7.app/label": "Mode (R7)",
110-
"pl7.app/table/visibility": "default",
110+
"pl7.app/table/visibility": "optional",
111111
"pl7.app/table/orderPriority": "1100"
112112
}
113113
},
@@ -262,9 +262,21 @@ scoresColumnSpecs := {
262262
"pl7.app/liabilities/cdrh3CompactnessLowConfidenceResidueFraction",
263263
"CDRH3 compactness low-conf fraction", "740"),
264264

265-
// R39 three-band threshold flags. Visible by default per R51.
266-
totalCdrLengthFlag: flagColumnSpec(
267-
"pl7.app/liabilities/totalCdrLengthFlag", "Total CDR length flag", "930"),
265+
// R39 three-band threshold flags. Visible by default per R51, except
266+
// `totalCdrLengthFlag` which is not in the spec's default-visible list
267+
// (totalCdrLength itself is "Hidden behind show all", and its flag
268+
// follows the raw metric).
269+
totalCdrLengthFlag: {
270+
valueType: "String",
271+
name: "pl7.app/liabilities/totalCdrLengthFlag",
272+
annotations: {
273+
"pl7.app/label": "Total CDR length flag",
274+
"pl7.app/isScore": "true",
275+
"pl7.app/description": _FLAG_DESCRIPTION,
276+
"pl7.app/table/visibility": "optional",
277+
"pl7.app/table/orderPriority": "930"
278+
}
279+
},
268280
pshFlag: flagColumnSpec(
269281
"pl7.app/liabilities/pshFlag", "PSH flag", "970"),
270282
ppcFlag: flagColumnSpec(

0 commit comments

Comments
 (0)