europa-tech-srl/arnold-trader-app
1
0
Fork 0
Code Issues 1 Pull requests 6 Projects Releases Packages Wiki Activity Actions

PIX-14540: remove remaining technical strategy stats buckets #625

Open
andrei wants to merge 1 commit from agent/devops/pix-14540-technical-bucket-cleanup-20260803 into master
pull from: agent/devops/pix-14540-technical-bucket-cleanup-20260803
merge into: europa-tech-srl:master
Conversation 1 Commits 1 Files changed 2 +42 -1
andrei commented 2026-08-03 09:57:47 +00:00
Owner
Copy link

PIX-14540 continuous Arnold loop.

Что исправлено:

  • _TECHNICAL_STRATEGIES теперь включает все technical buckets, которые уже запрещены как economic owner: exchange_sl_tp_close, futures_monitor, syncer_imported, unknown.
  • refresh_strategy_stats_for_user больше не пере-upsertит stale futures_monitor/technical rows и передаёт их в cleanup.

Live evidence до фикса:

  • production strategy_stats для user 2117705101: futures_monitor = 111 rows, 108 nonzero rows, summed PnL 2407.66.
  • economic strategy parity с raw trades после корректного side set: drift_rows=0, то есть проблема именно в stale technical bucket rows.

Проверка:

  • python3 -m pytest backend/tests/test_strategy_stats_updater.py -q -> 24 passed.
  • git diff --check -> clean.
  • node .gitnexus/run.cjs impact ... и detect-changes запускались, но локальный GitNexus index не смог отдать graph: No indexed repositories found; node .gitnexus/run.cjs analyze упал на DuckDB FTS/index extension.
PIX-14540 continuous Arnold loop. Что исправлено: - `_TECHNICAL_STRATEGIES` теперь включает все technical buckets, которые уже запрещены как economic owner: `exchange_sl_tp_close`, `futures_monitor`, `syncer_imported`, `unknown`. - `refresh_strategy_stats_for_user` больше не пере-upsertит stale `futures_monitor`/technical rows и передаёт их в cleanup. Live evidence до фикса: - production `strategy_stats` для user `2117705101`: `futures_monitor` = 111 rows, 108 nonzero rows, summed PnL 2407.66. - economic strategy parity с raw trades после корректного side set: `drift_rows=0`, то есть проблема именно в stale technical bucket rows. Проверка: - `python3 -m pytest backend/tests/test_strategy_stats_updater.py -q` -> `24 passed`. - `git diff --check` -> clean. - `node .gitnexus/run.cjs impact ...` и `detect-changes` запускались, но локальный GitNexus index не смог отдать graph: `No indexed repositories found`; `node .gitnexus/run.cjs analyze` упал на DuckDB FTS/index extension.
fix(stats): delete all technical strategy buckets
All checks were successful
Arnold Forgejo CI / backend-tests (pull_request) Successful in 12m25s
Details
Arnold Forgejo CI / frontend-audit (pull_request) Successful in 13s
Details
Arnold Forgejo CI / secret-scan (pull_request) Successful in 2s
Details
Forgejo Smoke Test / Smoke (pull_request) Successful in 1s
Details
1e0cf3f3fc
andrei commented 2026-08-03 10:04:11 +00:00
Author
Owner
Copy link

PR был закрыт без merge, поэтому я переоткрыл его как актуальный review path для PIX-14540. Локально повторно проверено: python3 -m pytest backend/tests/test_strategy_stats_updater.py -q -> 24 passed; git diff --check -> clean. Ждём arnold-reviewer и Forgejo CI для 1e0cf3f3.

PR был закрыт без merge, поэтому я переоткрыл его как актуальный review path для PIX-14540. Локально повторно проверено: `python3 -m pytest backend/tests/test_strategy_stats_updater.py -q` -> 24 passed; `git diff --check` -> clean. Ждём `arnold-reviewer` и Forgejo CI для `1e0cf3f3`.
All checks were successful
Arnold Forgejo CI / backend-tests (pull_request) Successful in 12m25s
Details
Arnold Forgejo CI / frontend-audit (pull_request) Successful in 13s
Details
Arnold Forgejo CI / secret-scan (pull_request) Successful in 2s
Details
Forgejo Smoke Test / Smoke (pull_request) Successful in 1s
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin agent/devops/pix-14540-technical-bucket-cleanup-20260803:agent/devops/pix-14540-technical-bucket-cleanup-20260803
git switch agent/devops/pix-14540-technical-bucket-cleanup-20260803

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff agent/devops/pix-14540-technical-bucket-cleanup-20260803
git switch agent/devops/pix-14540-technical-bucket-cleanup-20260803
git rebase master
git switch master
git merge --ff-only agent/devops/pix-14540-technical-bucket-cleanup-20260803
git switch agent/devops/pix-14540-technical-bucket-cleanup-20260803
git rebase master
git switch master
git merge --no-ff agent/devops/pix-14540-technical-bucket-cleanup-20260803
git switch master
git merge --squash agent/devops/pix-14540-technical-bucket-cleanup-20260803
git switch master
git merge --ff-only agent/devops/pix-14540-technical-bucket-cleanup-20260803
git switch master
git merge agent/devops/pix-14540-technical-bucket-cleanup-20260803
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
arnold-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
europa-tech-srl/arnold-trader-app!625
No description provided.