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

Update support for DJI air 3 drones #413

Draft
wants to merge 470 commits into
base: develop
Choose a base branch
from
Draft

Update support for DJI air 3 drones #413

wants to merge 470 commits into from

Conversation

Pradip-p
Copy link
Collaborator

@Pradip-p Pradip-p commented Dec 24, 2024

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Describe this PR

  • updated support for DJI air 3 drones

Screenshots

Please provide screenshots of the change.

Alternative Approaches Considered

Did you attempt any other approaches that are not documented in code?

Review Guide

Notes for the reviewer. How to test this change?

Checklist before requesting a review

  • πŸ“– Read the HOT Code of Conduct: https://docs.hotosm.org/code-of-conduct
  • πŸ‘·β€β™€οΈ Create small PRs. In most cases, this will be possible.
  • βœ… Provide tests for your changes.
  • πŸ“ Use descriptive commit messages.
  • πŸ“— Update any related documentation and include any relevant screenshots.

[optional] What gif best describes this PR or how it makes you feel?

Sujit and others added 30 commits November 26, 2024 19:20
Feat: Enhancements to Regulators Approval Flow on project creation
… unflyable task state (#357)

* feat: notify project creator when a task is marked as unflyable

* feat: upgrade drone-flightplan to >=0.3.2 and add email notification for unflyable task

* fix: added author email to send the email notification
bijayrauniyar0 and others added 23 commits December 11, 2024 15:39
…396)

* fix: calculate bbox for the drone image

* fix: calculate image footprints

* feat: get a list of images that contains a point
* refactor: remove useEffect and items to show state

* style: add css to hide spinner on input:number

* style: decrease width of select and input feilds

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* fix: add state image processing started to already image uploaded state

* fix: change updated time to be same as created at

* fix: add state for image processing to already present url

* refactor: add funcitonality to start image processing manually

* refactor: remove upload button

* style: add bg color for IMAGE_UPLOADED state

* fix: resolved image processing issues, state management for color, and dashboard listing

---------

Co-authored-by: Bijay Rauniyar <[email protected]>
Co-authored-by: Pradip-p <[email protected]>
* feat: auto add dem file if not provided by the user

for more information, see https://pre-commit.ci

* fix: scrapy signals issue with signals

for more information, see https://pre-commit.ci

* feat: add constants for dem radio data

* feat: add reducer and action ro set demTYpe

* feat: add dem type

* feat: auto add dem file if not provided by the user

for more information, see https://pre-commit.ci

* fix: scrapy signals issue with signals

for more information, see https://pre-commit.ci

* refactor: remove title from dem type switch

* fix: pre commit  fix

---------

Co-authored-by: Bijay Rauniyar <[email protected]>
* feat: implement api to save rotated flight plan

* refactor: remove type from flightPlan mutation

* refactor: update condition to show update image button

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
updates:
- [github.com/astral-sh/uv-pre-commit: 0.5.7 β†’ 0.5.9](astral-sh/uv-pre-commit@0.5.7...0.5.9)
- [github.com/astral-sh/ruff-pre-commit: v0.8.2 β†’ v0.8.3](astral-sh/ruff-pre-commit@v0.8.2...v0.8.3)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
#399)

* feat: add endpoints to process all task images of AOI

* reac: refactor process_assets_from_odm function to reduce redundancy

* fix: remove double s3_path from process_assets_from_odm

* fix: added all tasks images in odm processing

* fix: remove local setup of config & docker-compose

* fix: remove multple class for drone images processing & now handle from single class

* fix: refine drone image processing

* added tags in the odm webhook router endpoint

---------

Co-authored-by: Niraj Adhikari <[email protected]>
* Update faq.md

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* feat: setup the pytest

* feat: initial pytest setup for drone create

* feat: updated dependency overrides for login required

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* feat: updated test case for read drone

* feat: update user test routes & make dummy project data as fixtures

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
fix: update response for flight plan
updates:
- [github.com/astral-sh/uv-pre-commit: 0.5.9 β†’ 0.5.11](astral-sh/uv-pre-commit@0.5.9...0.5.11)
- [github.com/astral-sh/ruff-pre-commit: v0.8.3 β†’ v0.8.4](astral-sh/ruff-pre-commit@v0.8.3...v0.8.4)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@github-actions github-actions bot added enhancement New feature or request backend Related to backend code labels Dec 24, 2024
@@ -122,7 +122,8 @@ class HTTPStatus(IntEnum):


class DroneType(IntEnum):
DJI_MINI_4_PRO = 1
DJI_MINI_4_PRO = "DJI_MINI_4_PRO"
DJI_AIR_3 = "DJI_MINI_4_PRO"
Copy link
Member

Choose a reason for hiding this comment

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

Why are these both same ?

@Pradip-p Pradip-p changed the title Update support for DJJ air 3 drones Update support for DJI air 3 drones Dec 24, 2024
@Pradip-p Pradip-p self-assigned this Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants