Skip to content

__getitem__ with augmentedTensor #284

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 1 commit into from
Jan 12, 2023
Merged

__getitem__ with augmentedTensor #284

merged 1 commit into from
Jan 12, 2023

Conversation

ragier
Copy link
Contributor

@ragier ragier commented Nov 18, 2022

  1. fix getitem on augmentedTensor with augmented tensor as mask.
  2. reset name only if tensor aren't linearized (like in bbox unit test) else declass to classic tensor

@ragier
Copy link
Contributor Author

ragier commented Nov 18, 2022

#bug_compte_double

@ragier ragier marked this pull request as ready for review November 18, 2022 08:48
@ragier
Copy link
Contributor Author

ragier commented Nov 18, 2022

  1. Pass rotate image test
  2. Pass bbox test

@thibo73800 thibo73800 self-requested a review November 18, 2022 08:49
@thibo73800 thibo73800 requested a review from Data-Iab January 12, 2023 13:00
Data-Iab
Data-Iab approved these changes Jan 12, 2023
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.

seems ok, my tests passed as well

@Data-Iab Data-Iab merged commit 6a51940 into alobugdays Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

@thibo73800 thibo73800 Awaiting requested review from thibo73800

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ragier @Data-Iab