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

Bhavanatumma/ta2614748 #1294

Merged
merged 54 commits into from
Mar 7, 2024
Merged

Bhavanatumma/ta2614748 #1294

merged 54 commits into from
Mar 7, 2024

Conversation

BhavanaTumma
Copy link
Contributor

@BhavanaTumma BhavanaTumma commented Sep 22, 2023

Description

Add mlflow transformers conversion support for NLP models

Test

Validations: page

@BhavanaTumma BhavanaTumma requested a review from a team as a code owner September 22, 2023 15:34
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 22, 2023 15:34 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 22, 2023 15:35 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Sep 22, 2023

Test Results for training-model-mgmt-unittests

44 tests   44 ✅  8s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 3f9fbc4.

♻️ This comment has been updated with latest results.

@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 22, 2023 17:01 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 22, 2023 17:01 — with GitHub Actions Inactive
bhavanatumma added 2 commits September 27, 2023 13:25
…havanatumma/ta2614748

# Conflicts: Resolved
#	assets/training/model_management/src/azureml/model/mgmt/processors/transformers/convertors.py
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 27, 2023 08:02 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 27, 2023 08:02 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 27, 2023 08:04 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 27, 2023 08:04 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 30, 2023 08:06 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing September 30, 2023 08:06 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing October 15, 2023 16:28 — with GitHub Actions Inactive
@BhavanaTumma BhavanaTumma temporarily deployed to Testing October 15, 2023 16:28 — with GitHub Actions Inactive
@novaturient95
Copy link
Member

novaturient95 commented Mar 6, 2024

Code to add OSS flavor to the import model component

would be super helpful to add validation info as part of PR description. I understand we have done plenty but still good to have if we have to see the PR back again and confirm scenarios that were validated

novaturient95
novaturient95 previously approved these changes Mar 7, 2024
novaturient95
novaturient95 previously approved these changes Mar 7, 2024
@novaturient95 novaturient95 merged commit d1b2912 into main Mar 7, 2024
19 checks passed
@novaturient95 novaturient95 deleted the bhavanatumma/ta2614748 branch March 7, 2024 17:54
anushree1808 pushed a commit to anushree1808/azureml-assets that referenced this pull request Apr 23, 2024
* [bhavanatumma/ta2614748] - import model modification for OSS models

* [bhavanatumma/ta2614748] - Converting to MLFlow OSS transformers flavor

* [bhavanatumma/ta2614748] - Converting to MLFlow OSS transformers flavor

* [bhavanatumma/ta2614748] - Converting to MLFlow OSS transformers flavor

* [bhavanatumma/ta2614748] - Converting to MLFlow OSS transformers flavor - lint fix

* [bhavanatumma/ta2614748] - Converting to MLFlow OSS transformers flavor - v2, adding conda list approach

* [bhavanatumma/ta2614748] - lint fix

* [bhavanatumma/ta2614748] - review comments

* [bhavanatumma/ta2614748] - current progress

* [bhavanatumma/ta2614748] - current progress v2

* Merge branch 'main' of https://github.com/Azure/azureml-assets into bhavanatumma/ta2614748

# Conflicts: Resolved
#	assets/common/components/mlflow_model_local_validation/spec.yaml
#	assets/training/model_management/components/convert_model_to_mlflow/spec.yaml
#	assets/training/model_management/components/import_model/spec.yaml

* [bhavanatumma/ta2614748] - ref point

* [bhavanatumma/ta2614748] - ref point

* [bhavanatumma/ta2614748] - ref point v2

* [bhavanatumma/ta2614748] - added OSS and HF transformers support variable - model flavor

* [bhavanatumma/ta2614748] - lint fixes

* [bhavanatumma/ta2614748] - test fixes

* [bhavanatumma/ta2614748] - test fixes

* [bhavanatumma/ta2614748] - test fixes

* [bhavanatumma/ta2614748] - review comments and adding Whisper model

* [bhavanatumma/ta2614748] - review comments and adding Whisper model

* [bhavanatumma/ta2614748] - changing version

* [bhavanatumma/ta2614748] - changing versions

* [bhavanatumma/ta2614748] - changing signatures

* [bhavanatumma/ta2614748] - adding new env

* [bhavanatumma/ta2614748] - adding new task text2text

* [bhavanatumma/ta2614748] - adding new task text2text

* [bhavanatumma/ta2614748] - supporting t2t tasks

* [bhavanatumma/ta2614748] - whisper changes

* [bhavanatumma/ta2614748] - whisper changes + convertors

* [bhavanatumma/ta2614748] - fixing tests

* [bhavanatumma/summarization-oss] - version change

* [bhavanatumma/ta2614748] - lint fix

* [bhavanatumma/ta2614748] - signature fix

* [bhavanatumma/ta2614748] - signature fix

* pl

---------

Co-authored-by: bhavanatumma <[email protected]>
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.

5 participants