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

[civ2][5] move rllib fake_gpus + tf2-eager-tracing to civ2 #40031

Merged
merged 4 commits into from
Oct 11, 2023

Conversation

can-anyscale
Copy link
Collaborator

@can-anyscale can-anyscale commented Oct 2, 2023

Move rllib fake_gpus + tf2-eager-tracing to civ2. Nothing special, just move them to civ2.

Test:

  • CI

depends_on: rllibbuild
job_env: forge

- label: ":brain: rllib: benchmarks"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

move this down here, it looks nicer on the ui if things are sorted by size

@can-anyscale can-anyscale changed the base branch from can-civ-04 to master October 7, 2023 06:14
@can-anyscale can-anyscale changed the title [civ2][7] move rllib tf2-eager-tracing to civ2 [civ2][5] move rllib fake_gpus + tf2-eager-tracing to civ2 Oct 7, 2023
@@ -138,7 +138,6 @@ def get_commit_range():
# End of dry run.

skip_prefix_list = [
".buildkite/",
Copy link
Collaborator

Choose a reason for hiding this comment

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

why is this line remove?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Files in this list are basically skipped when determining which tests to run. Without this change, this PR does not trigger rllib tests, even though it changes the rllib.rayci.yaml file :)

@can-anyscale can-anyscale merged commit 3fb4e19 into master Oct 11, 2023
81 of 82 checks passed
@can-anyscale can-anyscale deleted the can-civ-05 branch October 11, 2023 00:58
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.

None yet

2 participants