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

[bug] #1798

Open
troglodite2 opened this issue Oct 17, 2022 · 0 comments
Open

[bug] #1798

troglodite2 opened this issue Oct 17, 2022 · 0 comments
Labels
bug for actual bugs (unsure? use type:question)

Comments

@troglodite2
Copy link

aliceVision_featureMatching throws SIGABRT in Photogrammetry and Camera Tracking pipeline

In the pipeline ImageMatchingMultiSFM takes Input A from StructureFromMotion and Input B from FeatureExtraction2.
CameraInit has the images extracted from KeyframeExtraction as its image group.
CameraInit2 has 25 images taken of an object within the video.

This produces imageMatches.txt of 20 to 25 lines depending on attribute settings.

With input to FeatureMatching2 being Input A or Input B "regionsPerView.getRegions(J, descType);" line 77 in ImageCollectionMatcher_generic.cpp provides a non-existant viewId (J in the above call).

This causes regionsPerView.getRegions() to attempt to access a value outside of the std map. This causes a SIGABRT

If we use the combined SfM instead of input A or input B, it seems to work.

To Reproduce

Create a new pipeline of type Photogrammetry and Camera Tracking
Add a KeyframeSelection node and point it at your video. Set attributes correctly for your camera.
Compute KeyframeSelection.
Import the output of KeyframeSelection as image group 1.
Import the images of something within the video as image group 2.
Verify the intrinisics are good
Compute through "StructureFromMotion2"

See abort messages.

Expected behavior
No aborts.
I suggest another assert() in RegionsPerView.hpp getRegions() which checks that viewId exists within _data.

Alicevision: * develop ecbfe1b Merge pull request #1277 from p12tic/optimize-voctree
Meshroom: develop 243c278 Merge pull request #1778 from alicevision/dev/lv/useLastPathForOpenSave

Screenshots
If applicable, add screenshots to help explain your problem.

Log
If applicable, copy paste the relevant log output (please embed the text in a markdown code tag "```" )

Desktop (please complete the following and other pertinent information):

Additional context
The video and images can be provided if unable to reproduce.

@troglodite2 troglodite2 added the bug for actual bugs (unsure? use type:question) label Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug for actual bugs (unsure? use type:question)
Projects
None yet
Development

No branches or pull requests

1 participant