Roofer v1.x overview
Open
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 197
- Forks
- 32
- PR merge metrics
- No merged PRs in 30d
Description
- Develop API
- Investigate more comprehensive API for individual steps from reconstruction pipeline (?)
- Refactor code. Focus on common datatypes, minimise code duplication etc.
- #11
- review attribute handling
- Improvements to handle very large input data without needing a separate pre-tiling step
- stream reconstruct output as CityJSON Sequences to disk or to another program
- Improve error handling roofer app
- keep track of which batch/tiles have completed during processing in a 'project file'. So that we can pickup immediately from a crashed tile if we restart (add a cli option to enable/disbale this behaviour). This is to handle crashes that do not throw exception. To identify a run use a hash of all input options/parameters.
- option to output intermediate data from crop phase for subsequent debugging/reconstruct
- misc features
- simplify parameters. Remove unnecesary ones, automatic tuning (eg based on point density) were possible.
- Automated testing
- Unit tests
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files, tests, or entry points are named; begin by breaking the roadmap into a single scoped item and locating the reconstruction pipeline or roofer CLI entry point. The listed goals range from API and refactoring to large-input streaming, restart state, error handling, parameter simplification, and unit tests, so completion is not defined for this issue as written.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- cli, computer-vision, data, testing
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 15/100