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
pythonPackages.face_recognition: 1.2.1 -> 1.2.2 #38361
Conversation
I think you should
|
@dotlambda that's not the problem. When running |
@Ma27 That's bad. Do you have an idea what could be causing the segfault? /cc @globin @GrahamcOfBorg build python2.pkgs.dlib python3.pkgs.dlib |
Don't have a spontaneous idea unfortunately. I can have a look at the diff of |
Success on x86_64-linux (full log) Attempted: python2.pkgs.dlib, python3.pkgs.dlib Partial log (click to expand)
|
I'm currently running |
Success on aarch64-linux (full log) Attempted: python2.pkgs.dlib, python3.pkgs.dlib Partial log (click to expand)
|
awesome, thanks! I updated the description as it is indeed somehow misleading regarding which build was broken by the last |
It seems like davisking/dlib@14cbb80 is causing this. However, I don't get a segfault but
|
The following fixes have been applied according to the changelog (https://github.com/ageitgey/face_recognition/releases) - Added the face_detection CLI command - Removed dependencies on scipy to make installation easier - Cleaned up KNN example and fixed a bug with drawing fonts to label detected faces in the demo Furthermore the maintainer switched to actual GIT tags for PyPI releases as discussed in ageitgey/face_recognition#417 and NixOS#37566
This reverts commit 7ec4573.
2237314
to
a770159
Compare
@dotlambda did you make any progress here or do you have any idea what could be causing this? |
No progress, I don't have time right now. Would you mind opening an upstream issue about this? Maybe they can help out. |
@dotlambda I can have a closer look tonight and file a bug on upstream. Will keep you updated in case there's anything relevant for us %) |
The current checklist for filing new patches suggests to run `nox-review wip` to ensure that the current change (e.g. package bumps) don't break package bumps: https://github.com/NixOS/nixpkgs/blob/e1dee4efcbffc72260025078bf8297a3b732509c/.github/PULL_REQUEST_TEMPLATE.md I experienced this behavior from time to time (see NixOS/nixpkgs#38513 or NixOS/nixpkgs#38361) which cost me some extra time to spot the reasons for additional breackage. Running `nox-review wip` can be quite time consuming, especially when bumping packages that trigger a mass-rebuild or a stdenv rebuild, *however* this patch helps to reduce the risk of additional breackage caused by unwanted package bumps.
Motivation for this change
pythonPackages.face_recognition
to the latest version to get several important fixes (see the commit message for further reference)pythonPackages.dlib
bump as its build is currently brokenThings done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)