Skip to content

[DON'T MERGE]#53575

Closed
kevin85421 wants to merge 1 commit intoray-project:masterfrom
kevin85421:20250604-devbox1-tmux0-ray7
Closed

[DON'T MERGE]#53575
kevin85421 wants to merge 1 commit intoray-project:masterfrom
kevin85421:20250604-devbox1-tmux0-ray7

Conversation

@kevin85421
Copy link
Copy Markdown
Member

@kevin85421 kevin85421 commented Jun 5, 2025

Why are these changes needed?

Related issue number

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

@kevin85421
test
4a271aa 
Signed-off-by: kaihsun <kaihsun@anyscale.com>
@kevin85421
Copy link
Copy Markdown
Member Author

kevin85421 commented Jun 5, 2025 


[INFO 2025-06-05 05:43:33,525] log.py: 40  Observed the following results:
--
  |  
  | success = 1
  | stage_0_time = 6.606377363204956
  | stage_1_time = 127.86416268348694
  | stage_1_avg_iteration_time = 12.786410593986512
  | stage_1_max_iteration_time = 13.211825132369995
  | stage_1_min_iteration_time = 11.402020692825317
  | stage_2_time = 170.02550077438354
  | stage_2_avg_iteration_time = 34.004983806610106
  | stage_2_max_iteration_time = 34.62005352973938
  | stage_2_min_iteration_time = 33.598843812942505
  | stage_3_creation_time = 2.1609303951263428
  | stage_3_time = 1874.8651015758514
  | stage_4_spread = 0.6071306871187769

kevin85421
kevin85421 commented Jun 5, 2025
View reviewed changes
Copy link
Copy Markdown
Member Author

@kevin85421 kevin85421 left a comment

Choose a reason for hiding this comment

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

explain which test has issue

  1. the benchmark is too niche
  2. clear object_ref.clear_address()
  3. clear entire object_ref and add a new field object_id in TaskArg.

@kevin85421 kevin85421 mentioned this pull request Jun 6, 2025
Closed
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 20, 2025

This pull request has been automatically marked as stale because it has not had
any activity for 14 days. It will be closed in another 14 days if no further activity occurs.
Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

@github-actions github-actions bot added the stale The issue is stale. It will be closed within 7 days unless there are further conversation label Jun 20, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 4, 2025

This pull request has been automatically closed because there has been no more activity in the 14 days
since being marked stale.

Please feel free to reopen or open a new pull request if you'd still like this to be addressed.

Again, you can always ask for help on our discussion forum or Ray's public slack channel.

Thanks again for your contribution!

@github-actions github-actions bot closed this Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

stale The issue is stale. It will be closed within 7 days unless there are further conversation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kevin85421