feat: API performance tracking and final infra integration#430
Open
RUFFY-369 wants to merge 14 commits intoNousResearch:mainfrom
Open
feat: API performance tracking and final infra integration#430RUFFY-369 wants to merge 14 commits intoNousResearch:mainfrom
RUFFY-369 wants to merge 14 commits intoNousResearch:mainfrom
Conversation
…ability Add EnsembleReward to atroposlib/envs/reward_fns/ with: - Multiple aggregation strategies: mean, median, min, majority_vote - Krippendorff's alpha inter-rater reliability metric - Per-item disagreement tracking for reward hacking detection - Full integration with RewardRegistry 17/17 tests passing.
…lity Add RewardNormalizer to atroposlib/envs/ with: - Welford's online algorithm for running mean/variance (no data storage) - Z-score and min-max normalization modes - Configurable reward clipping and warmup period - Checkpoint save/load support - Opt-in integration in BaseEnv via 3 new config fields - WandB metrics for normalization statistics 21/21 tests passing.
Add CurriculumScheduler to atroposlib/envs/ with: - EMA-based per-item difficulty tracking from reward signals - Quantile-based difficulty binning (configurable N bins) - Three sampling strategies: uniform, easy_first, competence_based - Competence-based strategy cites Platanios et al. 2019 - Opt-in integration in BaseEnv via 3 config fields - WandB metrics for difficulty distribution tracking - Checkpoint save/load support 22/22 tests passing.
Add APIPerformanceTracker to atroposlib/utils/ with: - Rolling window latency stats (p50/p95/p99) - Throughput monitoring (items/sec, requests/sec) - Compression ratio and payload size tracking - Automatic slow-request warnings - Integration in BaseEnv (init, _send_scored_data_to_api, wandb_log) 7/7 tests passing.
for more information, see https://pre-commit.ci
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Type
📝 General Information
Description
This PR unifies the entire stabilization sprint into
BaseEnv. The main addition is a high-resolutionAPIPerformanceTrackerto monitor throughput and latency bottlenecks between the trainer and inference nodes.It tracks rolling p50/p95/p99 latencies and
items_per_secthroughput. I also fixed a critical bug inBaseEnv.wandb_logwhere metrics from multiple servers were being overwritten instead of aggregated. This branch was the final one verified on Vast.ai and is confirmed compatible with downstreamhermes-agentenvironments.Related Issues
Type of Change
✅ Developer & Reviewer Checklist