gitadmin/foxhunt
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: remove --min-hold-bars from Argo workflow template

Browse Source 
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) <noreply@anthropic.com>
This commit is contained in:
jgrusewski
2026-04-18 01:28:08 +02:00
parent 5461915a97
commit 0a11185f1b
1 changed files with 0 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
infra/k8s/argo/train-template.yaml
View File

@@ -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}} || {