Skip to content

🐛 Fixed ray startup failure. - #1122

Merged
Phinease merged 2 commits into
developfrom
jpl/jpl_0822
Sep 3, 2025
Merged

Phinease merged 2 commits into
developfrom
jpl/jpl_0822

Conversation

@porkpink

@porkpink porkpink commented Sep 3, 2025 •

Copy link
Copy Markdown
Contributor

#1121 批量上传处理文件:
image

Copilot AI review requested due to automatic review settings September 3, 2025 08:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a Ray startup failure by creating a global RayConfig instance that can be accessed throughout the module. The fix addresses an issue where class methods init_ray_for_worker and init_ray_for_service were trying to use a ray_config variable that didn't exist.

  • Added a global ray_config instance of the RayConfig class at module level
  • Enables proper functionality of existing class methods that depend on this global instance

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread backend/data_process/ray_config.py Outdated
@codecov

codecov Bot commented Sep 3, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
backend/data_process/ray_config.py 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment thread backend/data_process/ray_config.py Outdated
@Phinease
Phinease merged commit cbb992c into develop Sep 3, 2025
5 of 6 checks passed
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.

4 participants