Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

Fix Hugging Face transformers integration no installing correct dependencies #347

Merged
merged 1 commit into from
Jul 30, 2021
Merged

Fix Hugging Face transformers integration no installing correct dependencies #347

merged 1 commit into from
Jul 30, 2021

Conversation

@markurtz
Copy link
Member

@markurtz markurtz commented Jul 30, 2021

No description provided.

@markurtz markurtz requested a review from a team July 30, 2021 15:10
@markurtz markurtz self-assigned this Jul 30, 2021
@markurtz markurtz requested review from horheynm, mgoin and natuan and removed request for a team July 30, 2021 15:10
bfineran
bfineran previously approved these changes Jul 30, 2021
View reviewed changes
Copy link
Contributor

@bfineran bfineran left a comment

Choose a reason for hiding this comment

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

LGTM, looks like the commit from #344 is in the commit history here for some reason

@markurtz
Copy link
Member Author

markurtz commented Jul 30, 2021

@bfineran rebased and force pushed up to get latest main

rahul-tuli
rahul-tuli approved these changes Jul 30, 2021
View reviewed changes
Copy link
Member

@rahul-tuli rahul-tuli left a comment

Choose a reason for hiding this comment

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

@markurtz markurtz merged commit 821fb5b into main Jul 30, 2021
markurtz added a commit that referenced this pull request Jul 30, 2021
@markurtz
[Cherry Pick] Update 0.6 release from latest (#348)
f5e6a2c 
* YOLOv5 Sparse Transfer Learning Command Fixes (#342)

* Update README Highlights for yolo and bert models (#344)

* Fix transformers not installing correct dependencies (#347)
kevinaer pushed a commit that referenced this pull request Aug 3, 2021
@markurtz @kevinaer
Fix transformers not installing correct dependencies (#347)
c1c118c
@markurtz markurtz deleted the transformers-fix branch September 1, 2021 11:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@natuan natuan Awaiting requested review from natuan

@mgoin mgoin Awaiting requested review from mgoin

@horheynm horheynm Awaiting requested review from horheynm

2 more reviewers

@bfineran bfineran bfineran approved these changes

@rahul-tuli rahul-tuli rahul-tuli approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@markurtz markurtz

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@markurtz @bfineran @rahul-tuli