Skip to content

feat: image stitching in celery #58

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
Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

feat: image stitching in celery #58

wants to merge 13 commits into from

Conversation

xmy314
Copy link
Contributor

@xmy314 xmy314 commented Oct 19, 2024 •
edited
Loading

Description

Image stitching algorithum through celery as to not block gcom operation.
Uses an ODM to do stitching.

Resolves #42 #43

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

How Has This Been Tested?

  • UBC forest satellite image works flawlessly.
  • pavement test images works with minor artifact.
  • drone image at 15 meter high works with minor artifact.
  • drone image at 30 meter high works flawlessly.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

xmy314 and others added 11 commits October 19, 2024 15:16
@xmy314
feat: image stitching in celery
a263d59 
added "cv2" and "stitching" dependency.
customized the image stitching offered by "stitching".
setup directory to store media files.
implemented test for image stitching.
@xmy314
Merge branch 'main' into feature-add_image_stitching
10983c5
@xmy314
feat: tuned such that drone images can be properly stitched.
1994f4e
@xmy314
Merge branch 'main' into feature-add_image_stitching
740683c
@21chanas3
Merge remote-tracking branch 'origin/main' into feature-add_image_sti…
752a60b 
…tching
@xmy314
Switched to using Open Drone Map.
62979f9 
more reliable.
takes more compute.
similiar quality.
extra dependency of odm through docker.
@xmy314
Updates test cases
3202dff 
remove ubc forest as odm only work with gps included image.
added dependency check for odm docker image.
@xmy314
breaking: intentionally wrong code to probe the behaviour of github a…
0352752 
…ction.
@xmy314
fixed the intentional logic. Should pass django test.
7c5f79a
@xmy314
duplicated readme.md into /src to pass docker test.
4abf5a1 
Commented out image stitching test as they do pass but would waste too much time if it runs every time. At the time of this commit, they pass.
@xmy314
ruff linter fix.
b6ea056
@xmy314 xmy314 marked this pull request as ready for review April 5, 2025 17:22
@xmy314 xmy314 requested a review from 21chanas3 April 5, 2025 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@21chanas3 21chanas3 Awaiting requested review from 21chanas3

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.

Develop Image Stitching Algorithm
2 participants
@xmy314 @21chanas3