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

Merge rocm-ci transformers changes from upstream #17

Closed
wants to merge 54 commits into from

Conversation

pnunna93
Copy link

@pnunna93 pnunna93 commented Oct 5, 2022

Merge upstream rocm-ci branch into this to be able to run ort HF workloads in DLM

microsoftopensource and others added 30 commits March 11, 2021 22:03
Updated option for training with onnxruntime training to '--ort'
Remove data based dependencies in T5 for ORT
Fix for ortmodule + ds config
raviskolli and others added 24 commits April 28, 2021 11:45
* add ort config for debertav2 model

* remove prints

* remove old commented code

* fix run style error

* add flake ignore comment

* trial to fix blackify format error
Remove model specific changes for BERT and DistilBERT
…module

hack to make roberta can run it ortmodule
Update to import ORTModule from torch_ort as torch-ort is now public
Don't need to explicitly use _original_module for ORT with this [PR](microsoft/onnxruntime#7847)
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Oct 5, 2022

The documentation is not available anymore as the PR was closed or merged.

@amathews-amd
Copy link
Collaborator

@pnunna93 , this PR is bringing in a lot of changes. Can we restrict it to a minimum that enables ORT in HF ?

@amathews-amd
Copy link
Collaborator

This PR is bringing ORT support to ROCm HF branch
cc: @jeffdaily @groenenboomj

@pnunna93
Copy link
Author

@pnunna93 , this PR is bringing in a lot of changes. Can we restrict it to a minimum that enables ORT in HF ?

@amathews-amd, Created a separate PR with just ORT changes:
#18

Closing this one as its not needed anymore

@pnunna93 pnunna93 closed this Oct 14, 2022
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.

9 participants