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

Fix OneOf and AugmentationsSequence handling of Mask, Keypoints, and Bounding Boxes; resolve apply_phase unpacking error #448

Merged
merged 3 commits into from
Feb 15, 2025

Conversation

AhmedDusuki
Copy link
Contributor

This pull request builds on #446 by @Travvy88, which addresses handling Mask, Keypoints, and Bounding Boxes for OneOf and AugmentationsSequence. However, the original PR causes an error when mask=None, keypoints=None, and bounding_boxes=None due to the AugmentationsPipeline's apply_phase function unpacking the result tuple (expecting 2 values but receiving 4 from AugmentationsSequence).

  • Modify AugmentationSequence to include additional information in the result tuple only when (mask is not None) or (keypoints is not None) or (bounding_boxes is not None).

Travvy88 and others added 2 commits October 29, 2024 09:16

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Caused an error when unpacking result tuple in augmentationpipeline's apply_phase.
@kwcckw
Copy link
Collaborator

kwcckw commented Feb 14, 2025

@AhmedDusuki Could you run the pre-commit and update the pull request again?

Install pre-commit: pip install pre-commit or conda env: conda install pre-commit

From the prompt, navigate into the augraphy repo, and run pre-commit run - - all-files. You may need to run several times until all checks pass.

@kwcckw kwcckw merged commit 9336756 into sparkfish:dev Feb 15, 2025
4 checks passed
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.

None yet

3 participants