fix(clp-package): Ensure at least one worker in default calculations (fixes #1509).#1547
Conversation
|
Warning Rate limit exceeded@junhaoliao has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 5 minutes and 12 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 for further information. 📒 Files selected for processing (1)
✨ Finishing touches
🧪 Generate unit tests (beta)
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 |
Description
Fixed the
_get_num_workers()method in controller.py to ensure at least 1 worker is returned, even on single-CPU systems. The error in the equation has long existed and was moved from start_clp.py in #1178 .Checklist
breaking change.
Validation performed
In WSL Ubuntu
Then changed the
processorssetting to1: https://learn.microsoft.com/en-us/windows/wsl/wsl-config#main-wsl-settings ; restarted wsl