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

Segformer model and recipe #754

Merged
merged 23 commits into from
Apr 17, 2023
Merged

Segformer model and recipe #754

merged 23 commits into from
Apr 17, 2023

Conversation

eran-deci
Copy link
Contributor

No description provided.

@dagshub
Copy link

dagshub bot commented Feb 28, 2023

@BloodAxe
Copy link
Contributor

BloodAxe commented Mar 9, 2023

Hi @eran-deci do you mind updating the PR to make the build passing? Looks like there is only black formatting that is left

@eran-deci eran-deci requested a review from BloodAxe March 10, 2023 11:02
Copy link
Collaborator

@ofrimasad ofrimasad left a comment

Choose a reason for hiding this comment

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

Very nice work.
Was this code taken from the original repo or another repo?
please make sure the code has Apache/BSD/MIT license
My comments are inline. we will fix some more general SG related comment in the future (just add the # TODO in the required places)

@eran-deci eran-deci requested a review from ofrimasad March 15, 2023 14:41
Copy link
Collaborator

@ofrimasad ofrimasad left a comment

Choose a reason for hiding this comment

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

LGTM

@ofrimasad ofrimasad merged commit e83af3d into master Apr 17, 2023
@ofrimasad ofrimasad deleted the segformer_model_and_recipe branch April 17, 2023 14:43
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.

3 participants