Skip to content
This repository has been archived by the owner on Sep 24, 2024. It is now read-only.

Update Ray job submission example in README #118

Merged
merged 2 commits into from
Sep 19, 2024

Conversation

dpoulopoulos
Copy link
Contributor

Commit 0975fb2 changes the CLI groups finetune and evaluate, but the Ray job submission example in the README file still uses the deprecated run command.

What's changing

Update the example to match the new API.

How to test it

Run the example script on a Ray cluster.

Copy link
Member

@veekaybee veekaybee left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@veekaybee
Copy link
Member

Will merge this on top of this PR merge, needed to pin a few dependencies #119

@veekaybee
Copy link
Member

Related version update: #120

@veekaybee
Copy link
Member

@dpoulopoulos Since we made a change in requirements here to satisfy failing tests, if you could also add a version bump in your PR in pyproject.toml so we can satisfy the semver check, this should be good to merge!

Commit 0975fb2 changes the CLI groups `finetune` and `evaluate`, but the
Ray job submission example in the README file still uses the deprecated
`run` command.

Update the example to match the new API.

Signed-off-by: Dimitris Poulopoulos <[email protected]>
@dpoulopoulos dpoulopoulos force-pushed the update-readme-ray-example branch from 713a498 to 4e53b16 Compare September 19, 2024 06:42
Add a version bump in `pyproject.toml` to satisfy the semver check.

Signed-off-by: Dimitris Poulopoulos <[email protected]>
@dpoulopoulos
Copy link
Contributor Author

dpoulopoulos commented Sep 19, 2024

@veekaybee I added a semver bump commit on top and rebased the whole branch on top of main so you can do a fast forward merge. I'm not sure if it makes sense to make it 0.14.0 or 0.13.1. Let me know if this is okay.

@veekaybee veekaybee merged commit 5a25a47 into mozilla-ai:main Sep 19, 2024
4 checks passed
@veekaybee
Copy link
Member

Perfect, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants