From 2ea89b1cb02a033201efd3bac7a9c045c9c05499 Mon Sep 17 00:00:00 2001 From: Kent Overstreet Date: Sat, 18 Apr 2026 22:17:55 -0400 Subject: [PATCH] amygdala: drop linear_aggregator, not in steering-vectors v0.12.2 Only mean/pca/logistic are exposed in the installed version. Co-Authored-By: Proof of Concept --- training/amygdala_training/train_with_library.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/training/amygdala_training/train_with_library.py b/training/amygdala_training/train_with_library.py index a349310..52506d0 100644 --- a/training/amygdala_training/train_with_library.py +++ b/training/amygdala_training/train_with_library.py @@ -41,7 +41,6 @@ from steering_vectors.aggregators import ( mean_aggregator, pca_aggregator, logistic_aggregator, - linear_aggregator, ) # Reuse corpus loader from the hand-rolled trainer. @@ -96,8 +95,6 @@ def _aggregator_from_name(name: str): return logistic_aggregator( sklearn_kwargs={"penalty": "l1", "solver": "liblinear", "C": 0.1} ) - if name == "linear": - return linear_aggregator() raise ValueError(f"unknown aggregator: {name}") @@ -116,7 +113,7 @@ def main() -> None: ap.add_argument( "--aggregator", default="mean", - choices=["mean", "pca", "logistic", "logistic_l1", "linear"], + choices=["mean", "pca", "logistic", "logistic_l1"], ) ap.add_argument("--max-negatives-per-positive", type=int, default=3) args = ap.parse_args()