添加了图像序列到点云的模块 - #3
Open
lhghfhdbdb wants to merge 26 commits into
Open
Conversation
Owner
|
同步上游 |
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new module to generate a fused point cloud from an input image sequence. Key changes include:
- New services and models under the "ply" directory to implement COLMAP-based point cloud generation.
- Updates to dependencies and project configuration (pyproject.toml and .python-version) to support the new module.
- Integration of the new API endpoint via a FastAPI router and updates to application dependencies.
Reviewed Changes
Copilot reviewed 20 out of 153 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| backend/pyproject.toml | Updated Python version requirements and added new dependencies. |
| backend/app/services/ply_service.py | Added PlyService which orchestrates COLMAP commands and point cloud creation. |
| backend/app/services/ply/models/* | New models and modules for Patchmatch-based depth estimation and evaluation. |
| backend/app/routers/ply_router.py | New FastAPI router exposing an endpoint for point cloud generation. |
| backend/app/main.py | Integrated the new ply router into the FastAPI application. |
| backend/app/dependencies.py | Updated service imports with the addition of PlyService. |
| Other files (datasets, colmap_input, drone_service, etc.) | Various supporting and utility changes for point cloud generation. |
Comments suppressed due to low confidence (1)
backend/.python-version:1
- The Python version is lowered from 3.12 to 3.8; please ensure all new code features are compatible with Python 3.8.
3.8
Comment on lines
+5
to
6
| from .ply_service import PlyService | ||
|
|
There was a problem hiding this comment.
There is a duplicate import of PlyService. Remove one of these redundant import statements to improve code clarity.
Suggested change
| from .ply_service import PlyService |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
输入文件夹,里面有一个名为images的文件夹,放着待处理的图像序列,运行项目后会在当前文件夹路径下生成fused.ply文件和处理过程产生的文件。