Skip to content

Comments

Package runtime assets properly#4

Merged
bxvtr merged 1 commit intomainfrom
package-runtime-assets
Feb 21, 2026
Merged

Package runtime assets properly#4
bxvtr merged 1 commit intomainfrom
package-runtime-assets

Conversation

@bxvtr
Copy link
Contributor

@bxvtr bxvtr commented Feb 21, 2026

Summary

Package all runtime assets into the Python distribution and cleanly separate runtime and dev dependencies in the Docker build.

Motivation

Ensure experiment configs and runtime files are available in the container without relying on workspace paths, improving reproducibility and supply chain correctness.

Type of Change

  • Bug fix
  • New feature
  • Performance improvement
  • Refactor
  • Documentation
  • Breaking change

Technical Details

  • Include all files under trading_runtime/ as package data via pyproject.toml
  • Install runtime and dev dependencies into separate prefixes in the build stage
  • Ship only runtime dependencies into the final image
  • Update Argo workflows to reference installed package paths

Determinism & Reproducibility

  • Change preserves deterministic behavior
  • Tests confirm reproducibility
  • No hidden state introduced

Performance Impact

  • No measurable impact
  • Benchmarks included
  • Performance improvement

Tests

  • Unit tests added/updated
  • Integration tests added/updated
  • Existing tests pass

Breaking Changes

Workflow config paths changed from workspace paths to installed package paths.

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated (if needed)

@bxvtr bxvtr self-assigned this Feb 21, 2026
@bxvtr bxvtr merged commit edc2787 into main Feb 21, 2026
1 check passed
@bxvtr bxvtr deleted the package-runtime-assets branch February 21, 2026 14:35
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