Closed wangzyphysics closed 1 month ago
[!WARNING]
Rate Limit Exceeded
@wangzyphysics has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 35 minutes and 43 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 5069cad5d7f71e0334b7a1da5c7640006bc54f9c and f82334d1a5f0ae2b39f13a38ee20a9390adaf351.
The recent changes introduce a new class CalyEvoStepMerge
and modify existing logic to support a new "merge" mode in the calypso
explore style. Adjustments were made across multiple files to accommodate this new mode, including changes to function signatures, imports, and test cases. Additionally, configuration updates and path modifications were implemented to enhance the functionality and flexibility of the system.
Files | Change Summary |
---|---|
dpgen2/entrypoint/submit.py |
Added import for CalyEvoStepMerge and modified make_concurrent_learning_op to handle expl_mode . |
dpgen2/op/caly_evo_step_merge.py |
Introduced new class CalyEvoStepMerge with methods for input/output signatures and execution. |
dpgen2/op/prep_caly_dp_optim.py |
Changed temp_dir assignment to work_dir / "opt_path_0" . |
dpgen2/op/prep_caly_input.py |
Added ntasks parameter and modified task_names type to BigParameter(List[str]) . |
dpgen2/superop/__init__.py |
Removed import of CalyEvoStep . |
dpgen2/superop/caly_evo_step.py |
Introduced wise_executor function to handle different expl_mode values and updated function calls accordingly. |
dpgen2/superop/prep_run_calypso.py |
Modified caly_evo_step_op parameter type and added expl_mode parameter. |
tests/op/test_prep_caly_dp_optim.py |
Updated path comparison in test_01_success to include "_0" . |
tests/test_caly_evo_step.py |
Removed shutil.rmtree("upload", ignore_errors=True) in tearDown method. |
tests/test_merge_caly_evo_step.py |
Added new test cases for CalyEvoStepMerge and related operations. |
tests/test_prep_run_caly.py |
Added tests for CalyEvoStepMerge and modified run_default_config dictionary. |
dpgen2/op/collect_run_caly.py |
Changed task_name parameter type to BigParameter(str) . |
examples/calypso/input.test.json |
Updated configuration settings, image versions, and atom configurations. |
sequenceDiagram
participant User
participant Submit
participant CalyEvoStepMerge
participant CalyEvoStep
User->>Submit: Trigger make_concurrent_learning_op
Submit->>Submit: Check expl_mode
alt expl_mode is "merge"
Submit->>CalyEvoStepMerge: Create instance
CalyEvoStepMerge-->>Submit: Instance created
else expl_mode is "default"
Submit->>CalyEvoStep: Create instance
CalyEvoStep-->>Submit: Instance created
end
Submit-->>User: Return instance based on expl_mode
sequenceDiagram
participant User
participant PrepRunCaly
participant WiseExecutor
participant CalyEvoStepMerge
participant CalyEvoStep
User->>PrepRunCaly: Call _prep_run_caly with expl_mode
PrepRunCaly->>WiseExecutor: Determine executor based on expl_mode
alt expl_mode is "merge"
WiseExecutor->>PrepRunCaly: Return None
PrepRunCaly->>CalyEvoStepMerge: Execute with None executor
CalyEvoStepMerge-->>PrepRunCaly: Execution result
else expl_mode is "default"
WiseExecutor->>PrepRunCaly: Return initialized executor
PrepRunCaly->>CalyEvoStep: Execute with initialized executor
CalyEvoStep-->>PrepRunCaly: Execution result
end
PrepRunCaly-->>User: Return execution result
[!TIP]
Early Access Features
- `gpt-4o` model for chat
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?
Attention: Patch coverage is 58.22785%
with 33 lines
in your changes are missing coverage. Please review.
Project coverage is 84.23%. Comparing base (
76c9657
) to head (f82334d
). Report is 1 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
one can choose mode from
default
andmerge
in run_explore_config/modeSummary by CodeRabbit
New Features
CalyEvoStepMerge
class to handle merging results from evolutionary steps.Improvements
input.test.json
to include new image versions and refined atom configurations.Bug Fixes
prep_caly_dp_optim.py
to correct directory structure.Tests
CalyEvoStepMerge
and other workflow operations.