Skip to content

[Setup] Fix dependency#39

Merged
comaniac merged 1 commit intoawslabs:mainfrom
chhzh123:fix_install
Feb 3, 2023
Merged

[Setup] Fix dependency#39
comaniac merged 1 commit intoawslabs:mainfrom
chhzh123:fix_install

Conversation

@chhzh123
Copy link
Contributor

@chhzh123 chhzh123 commented Feb 3, 2023

Description

This PR fixes the dependency issues. By default, users do not need to have DeepSpeed or HuggingFace Transformers to be installed.

Checklist

  • PR's title starts with a category (e.g. [Bugfix], [Model], [Tutorial], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

@chhzh123 chhzh123 requested a review from comaniac February 3, 2023 02:06
Copy link
Contributor

@comaniac comaniac left a comment

Choose a reason for hiding this comment

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

LGTM

@comaniac comaniac merged commit 3001c8a into awslabs:main Feb 3, 2023
@comaniac
Copy link
Contributor

comaniac commented Feb 3, 2023

Thanks @chhzh123

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.

2 participants