Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Nightly] fix microbenchmark scripts #26947

Merged
merged 2 commits into from
Jul 24, 2022

Conversation

scv119
Copy link
Contributor

@scv119 scv119 commented Jul 24, 2022

Signed-off-by: scv119 [email protected]

Why are these changes needed?

microbenchmarks failed complaining

   raise ValueError(f"Malformed address: {address}")
ValueError: Malformed address: 

fix it by set RAY_ADDRESS="local"

Related issue number

Checks

Signed-off-by: scv119 <[email protected]>
@scv119 scv119 marked this pull request as ready for review July 24, 2022 14:58
@@ -3413,7 +3413,7 @@

run:
timeout: 1800
script: OMP_NUM_THREADS=64 RAY_ADDRESS= python run_microbenchmark.py
script: OMP_NUM_THREADS=64 RAY_ADDRESS=auto python run_microbenchmark.py
Copy link
Contributor

Choose a reason for hiding this comment

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

Any ideas why this worked before?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

probably because of this pr 55a0f7b

@scv119 scv119 added the do-not-merge Do not merge this PR! label Jul 24, 2022
@scv119 scv119 removed the do-not-merge Do not merge this PR! label Jul 24, 2022
@scv119 scv119 merged commit acbab51 into ray-project:master Jul 24, 2022
Rohan138 pushed a commit to Rohan138/ray that referenced this pull request Jul 28, 2022
Signed-off-by: scv119 [email protected]

Why are these changes needed?
microbenchmarks failed complaining

   raise ValueError(f"Malformed address: {address}")
ValueError: Malformed address: 
this is due to 55a0f7b and fix it by set RAY_ADDRESS="local"

Signed-off-by: Rohan138 <[email protected]>
Stefan-1313 pushed a commit to Stefan-1313/ray_mod that referenced this pull request Aug 18, 2022
Signed-off-by: scv119 [email protected]

Why are these changes needed?
microbenchmarks failed complaining

   raise ValueError(f"Malformed address: {address}")
ValueError: Malformed address:
this is due to 55a0f7b and fix it by set RAY_ADDRESS="local"

Signed-off-by: Stefan van der Kleij <[email protected]>
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