feature_matching_utils_test fails on GPU builds without OpenSSL or curl

Aperta Adatta ai principianti
  • Qualcosa non va?
#5,020 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
85/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
cmake, cpp
Ambito
computer-vision, testing

Direzione di ricerca

Inizia dall'ultimo blocco di FeatureMatcherController.SetupPropagatesMatcherCreationErrors in controllers/feature_matching_utils_test, che viene eseguito solo con un dispositivo CUDA. Controlla kDefaultFlexGlueMatcherUri in src/colmap/feature/resources.h, che è vuoto senza supporto al download. Fai puntare matcher_model_path a un file mancante nel test, come già avviene per descriptor_model_path. È completato quando il test passa sia con sia senza supporto al download, e Setup() continua a fallire sul modello mancante.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

FeatureMatcherController.SetupPropagatesMatcherCreationErrors (in controllers/feature_matching_utils_test, added in #4957) fails on a machine with a CUDA device when COLMAP is built without download support.

The last block of the test runs only with a CUDA device. It sets flexglue->descriptor_model_path to a missing file, but it leaves matcher_model_path at its default, kDefaultFlexGlueMatcherUri. Without COLMAP_DOWNLOAD_ENABLED, that default is "" (src/colmap/feature/resources.h). So FlexGlueMatchingOptions::Check() fails on !matcher_model_path.empty(), and the FeatureMatcherController constructor throws before Setup() runs:

[flexglue.cc:491] Check failed: !matcher_model_path.empty()
C++ exception with description "[feature_matching_utils.cc:361] Check failed: matching_options_.Check() " thrown in the test body.

CI probably doesn't hit this because its runners either have no CUDA device or build with download support.

To Reproduce

  1. Build on a machine with a CUDA GPU, without the curl/OpenSSL dev packages. CMake then prints "Disabling download support (Curl/Crypto not found)".
  2. Run ctest -R controllers/feature_matching_utils_test --output-on-failure.
  3. See the error above.

Expected behavior
Check() passes, and Setup() fails on the missing model, as the test intends.

Suggested fix: point matcher_model_path at the missing file too, as the test already does for descriptor_model_path:

flexglue_options.flexglue->matcher_model_path =
    (data.test_dir / "missing_model.onnx").string();

Confirmed on the same machine and commit: built without download support, the test fails as above. After installing libcurl4-openssl-dev and libssl-dev, so that CMake prints "Enabling download support", the test passes unchanged (with the CUDA 12 runtime and cuBLAS that the bundled ONNX Runtime CUDA provider needs).

Screenshots
N/A

Environment:

  • OS: Ubuntu 24.04.3 LTS on WSL2; CUDA 13.3; NVIDIA GeForce RTX 5070 Ti (driver 610.88)
  • COLMAP Version: a branch on top of main 3c1a1b95 that does not touch FlexGlue, resources.h or this test; ONNX_ENABLED=ON
  • Capture Device: N/A
Lingua principale
C++
Stelle
12.9k
Fork
2.2k
Merge medio
1g 21h
PR unite (30g)
76

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di colmap/colmap

Tutte le issue di colmap/colmap

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.