Skip to content

Feature/compare scripts2#123

Closed
griwodz wants to merge 8 commits intodevelopfrom
feature/compareScripts2
Closed

Feature/compare scripts2#123
griwodz wants to merge 8 commits intodevelopfrom
feature/compareScripts2

Conversation

@griwodz
Copy link
Copy Markdown
Member

@griwodz griwodz commented Jan 5, 2021

Description

The test script runs VLFeat and PopSift on one or more images and creates the descriptors with orientations.
compareSiftFiles.cpp performs a brute force comparison of the output files and outputs sorted files and some distance computations.
These are then transformed bash and awk and rendered visually with gnuplot.

Features list

New binary that compares two files containing SIFT descriptors. VLFeat and Matlab syntax supported.
Scripts that compare VLFeat-compatible PopSift behaviour with original VLFeat behaviour. Requires new command line options of PR #114 before they can work.

Implementation remarks

The popsift-compareSiftFiles is fairly stand-alone. It uses boost, but not CUDA.

The scripts are based on bash, they depend on a lot of Unix command line tools, ImageMagick and Gnuplot. These should be seen as examples.

Carsten Griwodz added 8 commits December 29, 2020 11:02
[test] split compare functions
[test] remove binary
[test] rename file for Make executation
[test] minor code cleanup
[test] CMake for compareSiftFiles binary
[test] add compareDescriptors to cmake
[test] add popsift prefix to testing binary
[test] adapt scripts to new name
[test] new option dv, fix codacy bug
[test] typo
[test] move desc comparator to src
[test] CodeFactor fix
[test] update CHANGES.md
[test] fixed typo
[test] autofill some variables in compare script
[test] generate make helper for desc comparison
[test] boost program options in compareSiftFiles
[test] generated test files
[test] relative test script path
[test] convert test images
@griwodz griwodz closed this Jan 5, 2021
@griwodz griwodz deleted the feature/compareScripts2 branch January 5, 2021 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant