Skip to content

Fix atomic-energy loss prefactor documentation - #144

Closed
njzjz wants to merge 1 commit into
masterfrom
codex/issue-125
Closed

Fix atomic-energy loss prefactor documentation#144
njzjz wants to merge 1 commit into
masterfrom
codex/issue-125

Conversation

@njzjz

@njzjz njzjz commented Aug 23, 2026

Copy link
Copy Markdown
Member

Summary

  • keep separate start and limit atomic-energy documentation variables
  • attach those variables to the matching arguments
  • add regression assertions for both descriptions

Fixes #125.

Validation

  • python -m unittest -v tests.test_docgen
  • Ruff checks on changed files

AI attribution

Coding agent: Codex
Codex version: codex-cli 0.149.0
Model: gpt-5.6-sol
Reasoning effort: xhigh

Coding-Agent: Codex
Codex-Version: codex-cli 0.149.0
Model: gpt-5.6-sol
Reasoning-Effort: xhigh
@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@njzjz, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 59 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 624c19bc-022b-40f7-a44d-2007acae6a5e

📥 Commits

Reviewing files that changed from the base of the PR and between 66c7d8b and 27d1bdc.

📒 Files selected for processing (2)
  • tests/dpmdargs.py
  • tests/test_docgen.py

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

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Aug 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.14%. Comparing base (66c7d8b) to head (27d1bdc).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #144   +/-   ##
=======================================
  Coverage   75.14%   75.14%           
=======================================
  Files           9        9           
  Lines         853      853           
=======================================
  Hits          641      641           
  Misses        212      212           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@njzjz njzjz closed this Aug 23, 2026
@njzjz
njzjz deleted the codex/issue-125 branch August 23, 2026 19:18
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.

[Code scan] Correct atomic-energy loss docs in dpmdargs fixture

2 participants