Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

added and modified database from DBow3 #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tmako123
Copy link

I added and modified database from DBow3.
Check fbow_demo with argument
"orb ../../utils/images/image0.png ../../utils/images/image1.png ../../utils/images/image2.png ../../utils/images/image3.png"

@rmsalinas
Copy link
Owner

Hi, did you have the time to test the changes and verify it works? I'm planning to integrate them into the trunk

@ake020675
Copy link

hi ,i tested your changes and found errors caused by "export.h" in "Database.h" and "QueryResults.h". I changed it to be "flow_export.h" and all_build succeeded.

@ake020675
Copy link

Found other problems in "fbow_demo.cpp":
1 fbow::voc.save only support binary fromat;
2 database.save supprot "txt" or "yml" format because it runs with cv:Filestorage.
So i changed voc.saveToFile("small_voc.yml.gz") to voc.saveToFile("small_voc.bin"), and
db.save("small_db.yml.gz") to db.save("small_db.yml").
Now everything seems correct.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants