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
hdr-plus: init at 2020-10-29 #102922
hdr-plus: init at 2020-10-29 #102922
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just 2 minor nits 👍
pkgs/applications/graphics/hdr-plus/0001-Update-to-Halide-10.0.0.patch
Outdated
Show resolved
Hide resolved
4127b16
to
edd41c1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please fetch the upstream patch instead of putting it into nix?
Result of 1 package built:
|
Result of 1 package marked as broken and skipped:
|
I figured it'd be more stable to include the patch here instead of referring to my PR commit (that I may-or-may-not force-push over again at some point) but sure, will do. |
edd41c1
to
8a391ac
Compare
8a391ac
to
0191f9f
Compare
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 1 package built:
|
Result of 1 package marked as broken and skipped:
|
0191f9f
to
9d95646
Compare
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package built:
|
Motivation for this change
hdr-plus
is a burst photography pipeline implementation based on Google's HDR+ pipeline. More details can be found here. I'm packaging this for the Megapixels app (#98479) which greatly benefits from using thestack_frames
binary provided by this project.The included patch is required for building against Halide 10.0.0, it's a squashed version of my upstream PR timothybrooks/hdr-plus#70.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)