Skip to content

docs: Augmentations#310

Merged
giovannivolpe merged 10 commits intoDeepTrackAI:developfrom
Pwhsky:augmentations-examples
Mar 15, 2025
Merged

docs: Augmentations#310
giovannivolpe merged 10 commits intoDeepTrackAI:developfrom
Pwhsky:augmentations-examples

Conversation

@Pwhsky
Copy link
Copy Markdown
Collaborator

@Pwhsky Pwhsky commented Mar 3, 2025 •
edited
Loading

Changes:

  • Unit testing of Affine function.
  • Added more augmentations examples.
  • Import statements are now absolute.
  • Type hints

@giovannivolpe giovannivolpe marked this pull request as draft March 6, 2025 14:08
@giovannivolpe giovannivolpe marked this pull request as ready for review March 6, 2025 14:09
@giovannivolpe giovannivolpe requested a review from cmanzo March 6, 2025 14:09
@Pwhsky Pwhsky changed the title Augmentations examples Docs for Augmentations and Unit Tests Mar 11, 2025
cmanzo
cmanzo reviewed Mar 12, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@cmanzo cmanzo left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Ensure all classes support both Image and np.ndarray, including lists of each (list[Image] and list[np.ndarray]).

Update type hints in method signatures accordingly as well as in the Methods section of the docstring.

@Pwhsky
Copy link
Copy Markdown
Collaborator Author

Pwhsky commented Mar 14, 2025

Implemented feedback

@Pwhsky Pwhsky changed the title Docs for Augmentations and Unit Tests docs: Augmentations Mar 14, 2025
cmanzo
cmanzo approved these changes Mar 14, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@cmanzo cmanzo left a comment

Choose a reason for hiding this comment

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

looks good!

@giovannivolpe giovannivolpe merged commit 745dd4d into DeepTrackAI:develop Mar 15, 2025
25 checks passed
@Pwhsky Pwhsky deleted the augmentations-examples branch March 15, 2025 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mirjagranfors mirjagranfors Awaiting requested review from mirjagranfors

1 more reviewer

@cmanzo cmanzo cmanzo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Pwhsky @cmanzo @giovannivolpe