Closed njzjz closed 3 months ago
[!WARNING]
Rate limit exceeded
@njzjz has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 45 minutes and 58 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 9d001ca625a6326dece4cf6f37707e291ab60cd9 and 4c59892dc537611830fc317ebf69221d8afc2bbf.
The recent changes in deepmd/utils/data.py
and corresponding test files involve the deprecation of the trn_all_set
parameter in the DeepmdData
class, leading to a unified approach where all sets are trained and tested. This simplifies the logic for handling training directories and ensures that test sets are merged from all directories before shuffling. These updates address the issue where only the last set was tested, ensuring comprehensive testing across all sets.
Files/Modules | Change Summary |
---|---|
deepmd/utils/data.py |
Deprecated trn_all_set parameter, simplified logic for handling training directories, unified test set loading and shuffling. |
source/tests/tf/test_deepmd_data.py |
Updated tests to reflect changes in directory handling, test set merging, and batch size checks. |
source/tests/tf/test_deepmd_data_sys.py |
Adjusted batch sizes and array concatenations for coordinate and test data retrieval to align with new logic. |
sequenceDiagram
participant User
participant DeepmdData
participant TestSet
User->>DeepmdData: Initialize without trn_all_set
DeepmdData->>DeepmdData: Simplify training directory logic
User->>DeepmdData: Request test set
DeepmdData->>TestSet: Merge test sets from all directories
TestSet->>DeepmdData: Return merged and shuffled test set
DeepmdData->>User: Provide comprehensive test set
Objective | Addressed | Explanation |
---|---|---|
Only the last set of a LabeledSystem is tested in 'dp test' (#3860) | ✅ |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.66%. Comparing base (
674bad7
) to head (4c59892
). Report is 119 commits behind head on devel.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Fix #3860.
Remove
train_dirs
andtest_dir
inDeepmdData
.Summary by CodeRabbit
New Features
Bug Fixes
Tests