flathub / org.kde.kdenlive

https://flathub.org/apps/details/org.kde.kdenlive
17 stars 18 forks source link

Update nv-codec-headers to 12.1.14.0 #352

Closed Bleuzen closed 5 months ago

Bleuzen commented 6 months ago

This enables AV1 hardware encoding

flathubbot commented 6 months ago

Started test build 107279

flathubbot commented 6 months ago

Build 107279 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/90095/org.kde.kdenlive.flatpakref
flathubbot commented 6 months ago

Started test build 107794

flathubbot commented 6 months ago

Build 107794 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/90610/org.kde.kdenlive.flatpakref
flathubbot commented 6 months ago

Started test build 107805

flathubbot commented 6 months ago

Build 107805 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/90621/org.kde.kdenlive.flatpakref
travier commented 6 months ago

I've merged a few updates alongside this one. Can someone confirms that this builds works? Thanks

travier commented 6 months ago

Hum, we stop this update in https://github.com/flathub/org.kde.kdenlive/pull/259 so we might not be able to do it.

Bleuzen commented 5 months ago

Hum, we stop this update in #259 so we might not be able to do it.

I see so we need to decide if we favor supporting new features vs older GPUs. Given that even the most recent GPUs we would drop support for by this update are 11 years old already I question how important it is prevent progress for those

According to latest Steam hardware survey there is only one GPU still listed from the dropped ones which is still used by 0.38% of users - trending downwards

Meanwhile GPUs supporting AV1 which would benefit from this update are used by 12.3% (summed) of users

So while it is always sad dropping support for something - at this point I think it makes sense to update because it seems to affect a lot more people

but keep in mind that these numbers are taken from Steam and the Steam users are not exactly the same group as Kdenlive users - so it is not perfect but could help with the decision

CC @guihkx

guihkx commented 5 months ago

Thanks for pinging me, @Bleuzen.

I'm just an occasional contributor, though, so my opinion might not be that important here.

Anyway, I do still have a GTX 660, and this change will ultimately prevent me from using H.264 NVENC encoding, because the 12.x branch doesn't work with my GPU.

While I have my doubts about how ubiquitous the AV1 codec really is at the moment (especially when compared to H.264), I do understand and support the need of moving forward to newer codecs, and it'd be unreasonable to stay on the 11.x branch, forever.

All of that to say that I won't personally oppose or approve this PR, and I hope maintainers can use their best judgment.

travier commented 5 months ago

Appart from publishing two distinct Flatpaks, which is not ideal, we could also push this update to the main channel and keep the older libraries in the Beta remote to keep the Flatpak available for a while for users of old GPUs.

guihkx commented 5 months ago

That's a good solution, too! Unless it becomes a maintenance burden for you guys, of course.

By the way, this is not really a problem for me, because I know how to make my own build.

travier commented 5 months ago

bot, build

flathubbot commented 5 months ago

Queued test build for org.kde.kdenlive.

flathubbot commented 5 months ago

Started test build 111134

travier commented 5 months ago

I've updated this branch with a README with a note and more deps updates.

@Bleuzen Could you test the build here? Thanks

I've made https://github.com/flathub/org.kde.kdenlive/pull/360 to update the Beta version to the same content minus this update.

@guihkx Could you test the build there? Thanks

flathubbot commented 5 months ago

Started test build 111137

flathubbot commented 5 months ago

Build 111134 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/93949/org.kde.kdenlive.flatpakref
flathubbot commented 5 months ago

Build 111137 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/93952/org.kde.kdenlive.flatpakref
Bleuzen commented 5 months ago

@travier build 111137 works here

travier commented 5 months ago

Thanks!