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

[RLlib] Add comments to example files for templates. #48988

Merged
merged 5 commits into from
Dec 2, 2024

Conversation

ArturNiederfahrenhorst
Copy link
Contributor

Why are these changes needed?

In order to extract sections of the given files in a programmatic way, we inject comments in this PR.
This enables us to separate the declaration of the config and the imports from the rest.

@sven1977 sven1977 changed the title [RLlib] Add comments to example files for templates [RLlib] Add comments to example files for templates. Dec 2, 2024
env_to_module_connector=_make_env_to_module_connector,
)
.training(
learner_connector=_make_learner_connector,
train_batch_size_per_learner=4000, # 5000 on old yaml example
minibatch_size=128, # 500 on old yaml example
train_batch_size_per_learner=4000,
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Signed-off-by: Sven Mika <sven@anyscale.io>
Copy link
Contributor

@sven1977 sven1977 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @ArturNiederfahrenhorst , let's merge this.

@sven1977 sven1977 marked this pull request as ready for review December 2, 2024 09:31
@sven1977 sven1977 enabled auto-merge (squash) December 2, 2024 09:32
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Dec 2, 2024
@github-actions github-actions bot disabled auto-merge December 2, 2024 09:32
@sven1977 sven1977 added rllib RLlib related issues rllib-docs-or-examples Issues related to RLlib documentation or rllib/examples rllib-newstack labels Dec 2, 2024
@sven1977 sven1977 enabled auto-merge (squash) December 2, 2024 10:55
@sven1977 sven1977 merged commit f89aaf9 into ray-project:master Dec 2, 2024
6 of 7 checks passed
jecsand838 pushed a commit to jecsand838/ray that referenced this pull request Dec 4, 2024
Signed-off-by: Connor Sanders <connor@elastiflow.com>
dentiny pushed a commit to dentiny/ray that referenced this pull request Dec 7, 2024
ujjawal-khare pushed a commit to ujjawal-khare-27/ray that referenced this pull request Dec 17, 2024
Signed-off-by: ujjawal-khare <ujjawal.khare@dream11.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go add ONLY when ready to merge, run all tests rllib RLlib related issues rllib-docs-or-examples Issues related to RLlib documentation or rllib/examples rllib-newstack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants