From 0a11185f1b653b5188297f4f222771afceea2963 Mon Sep 17 00:00:00 2001 From: jgrusewski Date: Sat, 18 Apr 2026 01:28:08 +0200 Subject: [PATCH] fix: remove --min-hold-bars from Argo workflow template CLI arg was removed in cost-driven hold timing but Argo template still passed it, causing exit code 2 on H100 deploy. Co-Authored-By: Claude Opus 4.6 (1M context) --- infra/k8s/argo/train-template.yaml | 4 ---- 1 file changed, 4 deletions(-) diff --git a/infra/k8s/argo/train-template.yaml b/infra/k8s/argo/train-template.yaml index a3b451673..5b41ce3e4 100644 --- a/infra/k8s/argo/train-template.yaml +++ b/infra/k8s/argo/train-template.yaml @@ -63,7 +63,6 @@ spec: value: "0.25" - name: spread-ticks value: "1.0" - - name: min-hold-bars value: "5" - name: sanitizer value: "none" # "none", "memcheck", "racecheck", "synccheck" @@ -464,7 +463,6 @@ spec: --tick-size {{workflow.parameters.tick-size}} \ --spread-ticks {{workflow.parameters.spread-ticks}} \ --initial-capital {{workflow.parameters.initial-capital}} \ - --min-hold-bars {{workflow.parameters.min-hold-bars}} \ --data-dir /data/futures-baseline \ --mbp10-data-dir /data/futures-baseline-mbp10 \ --trades-data-dir /data/futures-baseline-trades \ @@ -564,7 +562,6 @@ spec: --tick-size {{workflow.parameters.tick-size}} \ --spread-ticks {{workflow.parameters.spread-ticks}} \ --initial-capital {{workflow.parameters.initial-capital}} \ - --min-hold-bars {{workflow.parameters.min-hold-bars}} \ --data-dir /data/futures-baseline \ --mbp10-data-dir /data/futures-baseline-mbp10 \ --trades-data-dir /data/futures-baseline-trades \ @@ -636,7 +633,6 @@ spec: --checkpoint-dir /workspace/output \ --output-dir /workspace/output/eval \ --initial-capital {{workflow.parameters.initial-capital}} \ - --min-hold-bars {{workflow.parameters.min-hold-bars}} \ --tx-cost-bps {{workflow.parameters.tx-cost-bps}} \ --tick-size {{workflow.parameters.tick-size}} \ --spread-ticks {{workflow.parameters.spread-ticks}} || {