-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add NFIR package #26698
base: master
Are you sure you want to change the base?
add NFIR package #26698
Conversation
694cf75
to
5b9bf70
Compare
recipes/nfir/all/patches/0001-find-OpenCV-provided-by-conan_toolchain.cmake.patch
Outdated
Show resolved
Hide resolved
recipes/nfir/all/patches/0002-enable-choose-between-shared-and-static-library.patch
Outdated
Show resolved
Hide resolved
recipes/nfir/all/patches/0004-set-NFIR-executable-name-to-NFIR_bin-tool.patch
Outdated
Show resolved
Hide resolved
@valgur, thank you for the helpful feedback. I'm curious, the PR was reviewed very quickly after being opened. However, I have two other PRs that have been waiting for review for several weeks/months with no activity. Is there any official way I can request a review for a PR? |
2713dbf
to
8a1f55c
Compare
1c615bb
to
f3dde3c
Compare
That's just how things work down here, I guess. 😐 I'm just a regular contributor, and don't have direct bearing on whether this gets merged or not, but I hope my feedback helps things move along a bit faster at least. |
0b55184
to
f7e39b6
Compare
Summary
NIST-made library and tool NFIR/0.2.0 conanized
Motivation
NFIR is a library (and tool) capable of upsampling or downsampling fingerprint images according to NIST standards. It can now be seamlessly integrated as a Conan dependency.
Details
Since the official NIST repository does not contain any Releases, I had to fork the repository and create a Release myself in order to link the Conan sources to it.