Skip to content

Tiny install numactl in Docker image - #62

Merged
fzyzcjy merged 1 commit into
mainfrom
feat/install_numactl
Oct 20, 2025
Merged

Tiny install numactl in Docker image#62
fzyzcjy merged 1 commit into
mainfrom
feat/install_numactl

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Oct 20, 2025

Copy link
Copy Markdown
Collaborator

useful for e.g. GB hardware w/ numa mode mem

@fzyzcjy
fzyzcjy merged commit 2e3fb15 into main Oct 20, 2025
1 check passed
fzyzcjy added a commit that referenced this pull request Mar 4, 2026
- Add DetectorContext dataclass in detectors/base.py with
  metric_store, mini_wandb, rank_placement, and job_status fields
- Change BaseFaultDetector.evaluate() signature to accept DetectorContext
- Update all 6 detector implementations to use ctx.*
- HangDetector/TrainingCrashDetector now check ctx.job_status directly
  instead of querying TRAINING_JOB_STATUS from MetricStore
- Controller builds DetectorContext each tick and passes it to the
  detector chain, eliminating the Prometheus scrape roundtrip delay

See discussion #62 in docs/3-discussions.md.
fzyzcjy added a commit that referenced this pull request Mar 19, 2026
JD-ETH pushed a commit to JensenFire/miles that referenced this pull request Apr 11, 2026
GuanxingLu pushed a commit to GuanxingLu/miles that referenced this pull request Apr 21, 2026
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