Skip to content
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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

tomas-krupa
Copy link
Contributor

@tomas-krupa tomas-krupa commented Feb 26, 2025

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.


@tomas-krupa tomas-krupa force-pushed the add-NFIR-recipe branch 5 times, most recently from 694cf75 to 5b9bf70 Compare February 26, 2025 12:34
@tomas-krupa
Copy link
Contributor Author

@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?

@tomas-krupa tomas-krupa force-pushed the add-NFIR-recipe branch 2 times, most recently from 2713dbf to 8a1f55c Compare February 26, 2025 16:24
@valgur
Copy link
Contributor

valgur commented Feb 26, 2025

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?

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.

@tomas-krupa tomas-krupa force-pushed the add-NFIR-recipe branch 4 times, most recently from 0b55184 to f7e39b6 Compare February 26, 2025 21:40
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.

2 participants