Skip to content

Dynamic cropping #188

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Jun 7, 2022
Merged

Dynamic cropping #188

merged 5 commits into from
Jun 7, 2022

Conversation

anhtu293
Copy link
Contributor

Dynamic Cropping

  • Possibility to crop an image to smaller fixed size image in the position we want. The crop position can be parsed by argument center which can be float or int.
  • If crop is out of image border, an error is triggered.

@anhtu293 anhtu293 marked this pull request as ready for review May 31, 2022 11:59
@anhtu293 anhtu293 requested a review from thibo73800 May 31, 2022 11:59
@anhtu293 anhtu293 self-assigned this May 31, 2022
thibo73800
thibo73800 requested changes May 31, 2022
View reviewed changes
@anhtu293 anhtu293 requested a review from thibo73800 May 31, 2022 15:11
Data-Iab
Data-Iab reviewed Jun 3, 2022
View reviewed changes
@anhtu293 anhtu293 requested a review from Data-Iab June 3, 2022 08:03
Data-Iab
Data-Iab reviewed Jun 3, 2022
View reviewed changes
@thibo73800 thibo73800 requested review from thibo73800 and removed request for thibo73800 June 3, 2022 13:44
@thibo73800
Copy link
Contributor

Will merge once @Data-Iab approved

Data-Iab
Data-Iab approved these changes Jun 3, 2022
View reviewed changes
Copy link
Collaborator

@Data-Iab Data-Iab left a comment

Choose a reason for hiding this comment

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

I think it's all good

@thibo73800 thibo73800 merged commit a8e3871 into master Jun 7, 2022
@thibo73800 thibo73800 deleted the transform-dynamic-crop branch June 7, 2022 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thibo73800 thibo73800 thibo73800 approved these changes

@Data-Iab Data-Iab Data-Iab approved these changes

Assignees

@anhtu293 anhtu293

Labels
alodataset
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@anhtu293 @thibo73800 @Data-Iab