Repository navigation
test(losses): improved composite loss tests without mocks - #513
k-chrispens wants to merge 1 commit into
Conversation
…ocks Weighted-sum tests now use weights that do not sum to one, so renormalizing by total weight would fail them. build_loss tests build the real 1vme density loss instead of monkeypatching build_single_loss, per the no-mocks guidance in AGENTS.md.
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 47 minutes. View limit details
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. Comment |
Weighted-sum tests now use weights that do not sum to one, so renormalizing by total weight would fail them. build_loss tests build the real 1vme density loss instead of monkeypatching build_single_loss, per the no-mocks guidance in AGENTS.md.