Simone Gasparini

The debug mode is covered here https://github.com/alicevision/popsift/blob/9dd7ae425340288b7b639b3fab42bc5ac1d529a2/CMakeLists.txt#L134-L148 It has been tested in vcpkg for every combination static/dynamic debug/release and there was no problem there https://github.com/microsoft/vcpkg/pull/10979 Could it be that it...

Just to be sure, you are getting this when building popsift, not when building your project that links to popsift? With your Conan recipe, are you using ninja to build...

It is actually broken when building from VS :-( Could you try to replace these lines from the cmake snippet above ```cmake if(CMAKE_BUILD_TYPE STREQUAL "Debug") set(PopSift_MVSC_LINKER "${PopSift_MVSC_LINKER}d") endif() list(APPEND CUDA_NVCC_FLAGS...

can you try to pass `-Dlz4_DIR:PATH=/usr/lib/pkgconfig/` at cmake step

px and py are offsets wrt the center of the image. I know it's not the standard way that everybody uses but here you have to add px and py...

Also for the focal length, again it is not a standard format as it is expressed in mm. To get it back in pixel you can use the formula ```...

As for the matrix, it should be stored in column-major order as per default in Eigen https://eigen.tuxfamily.org/dox/group__TopicStorageOrders.html So your R should be ok.

> also why is it `pxFocalLength = (focalLength / sensorWidth) *std::max(image().Width(), image().Height());` and not times the width for pxFocalLength and times the height for pyFocalLength? It's coming from here when...

just to be sure because I don't speak numpy, I was assuming that ``` M = K@R@q ``` is the matrix product of the matrices, right? so that we correctly...

> Thanks a lot! I got it just now. All the things you suggested were correct: > > * translating the principle point by adding width/2 and height/2 > *...