PointCloudLibrary / PointCloudLibrary/pcl
A number of test not being run on CI
Open
Nobody has claimed this yet.
kind: bug
module: ci
module: cmake
needs: testing
- Dominant language
- C++
- Stars
- 11.1k
- Forks
- 4.7k
- Avg merge
- 4d 10h
- Merged PRs (30d)
- 6
Description
While checking a build log I noticed some tests are not being built.
Something is off, likely on our cmake file.
-- The following subsystems will be built:
-- common
-- kdtree
-- octree
-- ml
-- segmentation
-- people
-- global_tests
-- tests_2d
-- tests_common
-- tests_features
-- tests_filters
-- tests_geometry
-- tests_io
-- tests_kdtree
-- tests_people
-- tests_octree
-- tests_search
-- tests_segmentation
-- tests_sample_consensus
-- tests_visualization
-- surface
-- registration
-- keypoints
-- tracking
-- recognition
-- stereo
-- apps
building:
|_ modeler
|_ point_cloud_editor
|_ 3d_rec_framework
|_ in_hand_scanner
|_ cloud_composer
-- simulation
-- outofcore
-- examples
-- tools
-- The following subsystems will not be built:
-- tests_keypoints: Requires keypoints.
-- tests_outofcore: Requires outofcore.
-- tests_recognition: Requires recognition.
-- tests_registration: Requires registration.
-- tests_surface: Requires surface.
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
Start with the linked Azure build log and the CMake configuration that determines subsystem prerequisites. Compare the listed tests that are not built with their dependency requirements and trace why they are omitted. Done means the intended test subsystems are built in CI and the configure output no longer omits them.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cmake, cpp
- Domain
- build-system, ci-cd, testing-qa
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 32/100