Skip to content

[AIR][Docs] Add BatchPredictor.from_checkpoint to docs - #32877

Merged
amogkam merged 3 commits into
ray-project:masterfrom
bveeramani:from-checkpoint
Feb 28, 2023
Merged

amogkam merged 3 commits into
ray-project:masterfrom
bveeramani:from-checkpoint

Conversation

@bveeramani

Copy link
Copy Markdown
Member

Why are these changes needed?

BatchPredictor.from_checkpoint is the most common way to create BatchPredictors, but it isn't documented. This PR adds the method to the docs.

Related issue number

Closes #32876

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 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 :(

Signed-off-by: Balaji Veeramani <balaji@anyscale.com>
Signed-off-by: Balaji Veeramani <balaji@anyscale.com>

@amogkam amogkam left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks!

Signed-off-by: Balaji Veeramani <balaji@anyscale.com>
@amogkam
amogkam merged commit 41eec15 into ray-project:master Feb 28, 2023
@bveeramani bveeramani mentioned this pull request Feb 28, 2023
7 tasks
ProjectsByJackHe pushed a commit to ProjectsByJackHe/ray that referenced this pull request Mar 21, 2023
…#32877)

BatchPredictor.from_checkpoint is the most common way to create BatchPredictors, but it isn't documented. This PR adds the method to the docs.

---------

Signed-off-by: Balaji Veeramani <balaji@anyscale.com>
Signed-off-by: Jack He <jackhe2345@gmail.com>
edoakes pushed a commit to edoakes/ray that referenced this pull request Mar 22, 2023
…#32877)

BatchPredictor.from_checkpoint is the most common way to create BatchPredictors, but it isn't documented. This PR adds the method to the docs.

---------

Signed-off-by: Balaji Veeramani <balaji@anyscale.com>
Signed-off-by: Edward Oakes <ed.nmi.oakes@gmail.com>
peytondmurray pushed a commit to peytondmurray/ray that referenced this pull request Mar 22, 2023
…#32877)

BatchPredictor.from_checkpoint is the most common way to create BatchPredictors, but it isn't documented. This PR adds the method to the docs.

---------

Signed-off-by: Balaji Veeramani <balaji@anyscale.com>
elliottower pushed a commit to elliottower/ray that referenced this pull request Apr 22, 2023
…#32877)

BatchPredictor.from_checkpoint is the most common way to create BatchPredictors, but it isn't documented. This PR adds the method to the docs.

---------

Signed-off-by: Balaji Veeramani <balaji@anyscale.com>
Signed-off-by: elliottower <elliot@elliottower.com>
ProjectsByJackHe pushed a commit to ProjectsByJackHe/ray that referenced this pull request May 4, 2023
…#32877)

BatchPredictor.from_checkpoint is the most common way to create BatchPredictors, but it isn't documented. This PR adds the method to the docs.

---------

Signed-off-by: Balaji Veeramani <balaji@anyscale.com>
Signed-off-by: Jack He <jackhe2345@gmail.com>
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.

[AIR] BatchPredictor.from_checkpoint isn't documented

2 participants