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
obs-studio-plugins.obs-backgroundremoval: 0.5.16 -> 1.1.13 #230775
base: master
Are you sure you want to change the base?
Conversation
7c79881
to
1f61c68
Compare
044286e
to
0e221ce
Compare
Does this still need to be a draft? |
Still pending #228093 |
#260745 seems promising |
I had trouble finding the most recent version of the plugin that builds with onnx 1.15.1 (current in unstable). I've been trying for well over an hour. |
Maybe we could ask the plugin developers to provide a way to build it in CPU-only mode, which would not require tensorrt_provider_factory.h to build, if that is not already an option. |
25daf81
to
a26b580
Compare
e39a7e4
to
136d2f2
Compare
onnx seems fine. onnxruntime fails to build though. should be fixed by #258392 according to #281011 (comment) |
136d2f2
to
07063e8
Compare
Ok this builds now. PS: On Ubuntu, that model is located at /usr/share/obs/obs-plugins/obs-backgroundremoval/models/mediapipe.onnx PSS: also getting |
@spikespaz I opened an issue on the developers’ repo at occ-ai/obs-backgroundremoval#544. |
07063e8
to
e232059
Compare
e232059
to
f0ca9e0
Compare
a21c551
to
4dd8fb2
Compare
Fixed the installation path by removing the extra /usr in the install cmake line. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3681 |
pkgs/applications/video/obs-studio/plugins/obs-backgroundremoval/default.nix
Outdated
Show resolved
Hide resolved
buildPhase = '' | ||
cd .. | ||
cmake --build build_x86_64 --parallel | ||
''; | ||
|
||
installPhase = '' | ||
cmake --install build_x86_64 --prefix "$out" | ||
''; |
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't we use cmakeFlags and the normal phases?
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.
cmakeFlags will apply to the normal buildPhase, right?
How to specify flags for installPhase?
Where is the documentation about this?
pkgs/applications/video/obs-studio/plugins/obs-backgroundremoval/default.nix
Outdated
Show resolved
Hide resolved
Is this PR still relevant or do we need a new one? |
Sorry I forgot this was pending. In my head this was done and merged. Let me check the review comments. |
…al/default.nix Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
33a9d03
to
7753a8e
Compare
7753a8e
to
c136052
Compare
Description of changes
occ-ai/obs-backgroundremoval@v0.5.16...1.1.13
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Requires #228093 (includes tensorrt_provider_factory.h to build)