Skip to content

fix: trigger downstream experiment validation#31

Merged
Pigbibi merged 1 commit intomainfrom
fix/experiment-validation-trigger
Apr 1, 2026
Merged

fix: trigger downstream experiment validation#31
Pigbibi merged 1 commit intomainfrom
fix/experiment-validation-trigger

Conversation

@Pigbibi
Copy link
Copy Markdown
Collaborator

@Pigbibi Pigbibi commented Apr 1, 2026

Summary

  • run upstream experiment validation on the self-hosted runner where Binance history downloads already work
  • toggle a downstream experiment-validation label so BinancePlatform can start validation from the issue event path
  • keep the downstream workflow-dispatch attempt as a non-fatal fallback

Testing

  • python3 -m unittest tests/test_prepare_experiment_validation.py tests/test_render_experiment_validation_summary.py tests/test_post_experiment_validation_comment.py tests/test_experiment_validation_workflow_config.py tests/test_monthly_optimization_planner_workflow_config.py
  • uvx ruff check scripts tests
  • ruby -e 'require "yaml"; [".github/workflows/experiment_validation.yml", ".github/workflows/monthly_optimization_planner.yml"].each { |p| YAML.load_file(p); puts "parsed #{p}" }'

@Pigbibi Pigbibi merged commit 91fefc6 into main Apr 1, 2026
1 check passed
@Pigbibi Pigbibi deleted the fix/experiment-validation-trigger branch April 1, 2026 21:41
@Pigbibi Pigbibi restored the fix/experiment-validation-trigger branch April 1, 2026 21:44
@Pigbibi Pigbibi deleted the fix/experiment-validation-trigger branch April 1, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant