Skip to content

Conversation

@ji-huazhong
Copy link
Contributor

@ji-huazhong ji-huazhong commented Oct 29, 2023

What does this PR do?

Part of #25654 (comment)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline,
    Pull Request section?
  • Was this discussed/approved via a Github issue or the forum? Please add a link
    to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the
    documentation guidelines, and
    here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

cc @ydshieh

@ji-huazhong
Copy link
Contributor Author

ji-huazhong commented Oct 29, 2023

verified with Mac-Studio

image image image

@ji-huazhong ji-huazhong marked this pull request as draft October 29, 2023 04:34
@ji-huazhong ji-huazhong force-pushed the device-agnositc-pipelines-testing branch 3 times, most recently from 5f90dad to 025994f Compare October 29, 2023 05:31
@ji-huazhong ji-huazhong marked this pull request as ready for review October 29, 2023 05:47
@amyeroberts
Copy link
Contributor

cc @ydshieh

@ji-huazhong ji-huazhong marked this pull request as draft October 30, 2023 08:08
@ji-huazhong ji-huazhong force-pushed the device-agnositc-pipelines-testing branch 3 times, most recently from cc7c4bd to 35eff72 Compare October 30, 2023 09:27
@ji-huazhong ji-huazhong marked this pull request as ready for review October 30, 2023 09:35
@ji-huazhong ji-huazhong force-pushed the device-agnositc-pipelines-testing branch from 35eff72 to 9f34d7b Compare October 31, 2023 02:12
@ji-huazhong
Copy link
Contributor Author

ji-huazhong commented Oct 31, 2023

@ydshieh Marking this ready for review :-)

Copy link
Contributor

@amyeroberts amyeroberts 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 working on this and improving our tests! ❤️

Happy to merge once @ydshieh approves

Copy link
Collaborator

@ydshieh ydshieh left a comment

Choose a reason for hiding this comment

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

Great! I would like to run them before merge. Will update here.

model="hf-internal-testing/tiny-random-distilbert",
framework="pt",
device=torch.device("cpu"),
device=torch_device,
Copy link
Collaborator

Choose a reason for hiding this comment

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

not sure why it was hardcoded to cpu previously

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@ydshieh
Copy link
Collaborator

ydshieh commented Oct 31, 2023

test run results look good, so merge! Thanks again @statelesshz

@ydshieh ydshieh merged commit f53041a into huggingface:main Oct 31, 2023
@ji-huazhong ji-huazhong deleted the device-agnositc-pipelines-testing branch November 1, 2023 06:27
EduardoPach pushed a commit to EduardoPach/transformers that referenced this pull request Nov 19, 2023
* device agnostic pipelines testing

* pass torch_device
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