Hourly sampler: aggregate smaller jobs

[rbx]

No description provided.

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request introduces aggregation support for the hourly sampler by adding new methods to compute and sample aggregated hourly job templates, separates queue and backlog metrics tracking throughout the codebase, and adds a new test for validating the aggregated sampling functionality.

Changes

Cohort / File(s)	Summary
Test Infrastructure .github/workflows/tests.yml, test/run_all.py, test/test_sampler_hourly_aggregated.py	Added new hourly aggregated sampler test step to CI workflow and test suite. New test module exercises precalculate_hourly_templates and sample_aggregated methods with configurable parameters and consistency validation.
Sampler Aggregation src/sampler_hourly.py, src/workload_generator.py	Introduced aggregation workflow: precalculate_hourly_templates() converts sub-hour jobs by resource profile into hourly templates; sample_aggregated() samples these templates with proportional scaling and probabilistic rounding. Updated workload generator to use aggregated sampling and duration_hours field directly.
Metrics Refactoring src/baseline.py, src/callbacks.py, src/metrics_tracker.py, src/environment.py	Separated combined queue+backlog tracking into distinct metrics: queue_only_size (active jobs with duration > 0) and backlog_size. Added corresponding max_backlog_size metrics across timeline and episode scopes for both agent and baseline. Consolidated metric logging at episode boundaries.
Documentation CLAUDE.md	Expanded Samplers section with detailed descriptions of Duration, Jobs, and Hourly Samplers. Added explicit "Aggregation support" section documenting precalculation, binning, template generation, and aggregated sampling behavior.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~35 minutes

Possibly related PRs

• Add CI tests #8: Modifies CI test orchestration and adds sampler test steps alongside this PR's test infrastructure changes.

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 64.29% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	No pull request description was provided by the author, making it impossible to evaluate whether the description relates to the changeset.	Add a pull request description explaining the purpose and impact of the aggregation feature for the hourly sampler.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Hourly sampler: aggregate smaller jobs' directly reflects the main changes in the PR, which introduce aggregation functionality to the HourlySampler class for combining sub-hour jobs into hourly templates.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch hourly_aggregation

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}