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

vulkan-hdr-layer-kwin6: init at 0-unstable-2024-10-19 #345805

Merged
merged 2 commits into from
Dec 7, 2024

Conversation

d4rk
Copy link
Contributor

@d4rk d4rk commented Oct 1, 2024

Enables HDR support for Vulkan clients (such as mpv) on kwin6.

https://github.com/Zamundaaa/VK_hdr_layer

This is likely to be a temporary solution until Wayland upstream adopts the necessary protocols, however that has been in the works for many years.

For the time being, this provides a path for users to experiment with HDR, share feedback and hopefully make progress.

Arch provides a similar package:
https://aur.archlinux.org/packages/vk-hdr-layer-kwin6-git

Example usage after installing the layer:

ENABLE_HDR_WSI=1 mpv \
    --hwdec=auto-safe --vo=gpu-next --target-colorspace-hint \
    --gpu-api=vulkan --gpu-context=waylandvk \
    <filename>

To enable Vulkan loader logs:

VK_LOADER_DEBUG=error,warn,info ENABLE_HDR_WSI=1 mpv \
    --hwdec=auto-safe --vo=gpu-next --target-colorspace-hint \
    --gpu-api=vulkan --gpu-context=waylandvk \
    <filename>

PS: The latest version of gamescope doesn't need Vulkan for HDR.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@github-actions github-actions bot added the 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` label Oct 1, 2024
@NixOSInfra NixOSInfra added the 12. first-time contribution This PR is the author's first one; please be gentle! label Oct 2, 2024
@ofborg ofborg bot added 8.has: package (new) This PR adds a new package 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1 labels Oct 2, 2024
@nixos-discourse
Copy link

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/4632

@FliegendeWurst
Copy link
Member

Would be good to squash the commits

@d4rk
Copy link
Contributor Author

d4rk commented Nov 3, 2024

Would be good to squash the commits

Done!

@d4rk d4rk force-pushed the vulkan-hdr-layer branch from ae237fb to 1b73ef5 Compare November 3, 2024 00:15
@d4rk d4rk force-pushed the vulkan-hdr-layer branch 3 times, most recently from 4c9e24b to 4224ee0 Compare November 3, 2024 23:55
@d4rk d4rk changed the title vulkan-hdr-layer-kwin6: init at 0.1.0 vulkan-hdr-layer-kwin6: init at 0-unstable-2024-10-19 Nov 3, 2024
d4rk added 2 commits November 11, 2024 17:21
https://github.com/Zamundaaa/VK_hdr_layer

Enables HDR support for Vulkan clients (such as mpv) on kwin6.

This is likely to be a temporary solution until Wayland upstream
adopts the necessary protocols, however that has been in the works for
many years:

https://gitlab.freedesktop.org/wayland/wayland-protocols/-/merge_requests/14

For the time being, this provides a path for users to experiment with
HDR, share feedback and hopefully make progress.

Arch provides a similar package:
https://aur.archlinux.org/packages/vk-hdr-layer-kwin6-git
@FliegendeWurst FliegendeWurst added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Nov 13, 2024
@d4rk
Copy link
Contributor Author

d4rk commented Nov 16, 2024

Thanks for the review @FliegendeWurst! Just wondering if this is ready to be merged and if so, when that might happen.

@FliegendeWurst
Copy link
Member

In my opinion it is ready. All that's left is that someone with permission merges.

@FliegendeWurst FliegendeWurst added the needs_merger (old Marvin label, do not use) label Nov 16, 2024
@kevincox kevincox merged commit b139d3d into NixOS:master Dec 7, 2024
31 checks passed
@d4rk d4rk deleted the vulkan-hdr-layer branch December 7, 2024 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1 11.by: package-maintainer This PR was created by the maintainer of the package it changes 12.approvals: 1 This PR was reviewed and approved by one reputable person 12. first-time contribution This PR is the author's first one; please be gentle! needs_merger (old Marvin label, do not use)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants