Skip to content

feat: implement image orientation annotation and importing logic#1549

Closed
areebahmeddd wants to merge 12 commits intoopenfoodfacts:mainfrom
areebahmeddd:issues/971/image-rotate
Closed

feat: implement image orientation annotation and importing logic#1549
areebahmeddd wants to merge 12 commits intoopenfoodfacts:mainfrom
areebahmeddd:issues/971/image-rotate

Conversation

@areebahmeddd
Copy link
Member

Summary

  • Added ImageOrientationImporter to generate insights for images requiring rotation (i.e., when orientation is not "up" and confidence is ≥95%)
  • Added ImageOrientationAnnotator to process approved insights by applying the appropriate rotation to images.
  • Added unit and integration tests to verify functionality of both components.

Done:
-Unit tests all green.

Todo:
-Investigate integration test failures (database schema issues??)

Part of

Issue #971

Copy link
Member

@alexgarel alexgarel left a comment

Choose a reason for hiding this comment

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

@areebahmeddd good work, you are on the right track ! Please see my comments.

@alexgarel
Copy link
Member

@areebahmeddd, see #971 (comment)

Sorry for I misguide you to a fix that was not the right one.

@areebahmeddd
Copy link
Member Author

Ah i see! No problem 😄 Currently out of station for a while, will address the updated requirements over the weekend!

@areebahmeddd areebahmeddd closed this by deleting the head repository May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants