deep-image-matching icon indicating copy to clipboard operation
deep-image-matching copied to clipboard

small bug(?) in exporting COLMAP database

Open oUp2Uo opened this issue 11 months ago • 4 comments

Hi, recently I am trying SFM disambiguation which will need to edit COLMAP database, and I find that the exported COLMAP database has different columns when using different images. Sometimes qvec and tvec is exited (this is normal situation). But occasionally, there are no these two columns, while running the same commands but using different images. Although these values are all null, but some other programs will read these columns when loading COLMAP database. I try to add qvec and tvec manually and it works.

qvec_tvec

no_qvec_tvec

I have tried to check the source code, but still could not understand why qvec and tvec columns are gone. Maybe is the difference between calling add_matches and add_raw_matches? https://github.com/3DOM-FBK/deep-image-matching/blob/master/src/deep_image_matching/io/h5_to_db.py

Thank you.

oUp2Uo avatar Jan 10 '25 03:01 oUp2Uo

Hi, thanks for reporting and sorry for the late reply. Did you find the source of the issue? The databases you are showing are generated both from DIM or one of them is just from COLMAP?

lcmrl avatar Feb 06 '25 08:02 lcmrl

Hi @lcmrl ,

Thanks for the reply. As I said in the last post, the databases were all generated by DIM (using different images), but some result would have qvec and tvec, and some would not. I am not sure about the reason. Maybe is the difference between calling add_matches and add_raw_matches.

I just check the database manually using sqlitestudio, and add these columns if there are no qvec and tvec.

oUp2Uo avatar Feb 07 '25 01:02 oUp2Uo

Ok thanks, I am trying to reproduce the problem and to solve it. This bug should not affect the performance, but I keep the issue open if someone else encounter problems on that

lcmrl avatar Feb 10 '25 16:02 lcmrl

Hi @lcmrl , Yes, that is why I wrote 'small bug' in the title. : )

oUp2Uo avatar Feb 11 '25 09:02 oUp2Uo