Kacper Michajłow

Results 593 comments of Kacper Michajłow

> If is a non-PQ video and `target-trc` is not set, `video-target-params/gamma` is `auto`, is this the expected behavior? @arm64-v9a: You can test with https://code.videolan.org/videolan/libplacebo/-/merge_requests/557 and this diff ``` diff...

> Although not relevant to this PR, I think there are too many spaces next to the HDR metadata. After looking at it indeed it was bit much. Reduced a...

> so why not subtitles stats too? There is not much interesting things to be shown about current selected subtitle. Well maybe, language and type. It is possible, but I...

> I feel it would be more aesthetically pleasing to put Video Out under Video? I see your point, but as already pointed Video and Audio should be together. Else...

> How about adding Picture Type and Frame number? Very useful for anime screenshots It may sound strange from me, because I added to this page a lot of things...

> * Consider renaming "Video Out" to just Display or something Changed to `Display:`, > * Maybe only list specified and estimated display fps and file fps if they differ?...

@llyyr @arm64-v9a @Hrxn Ok, I'm out of ideas for now. Take a look and let me know what do you think now.

> There's some refactoring you can do but I believe they should be their own PR. Yes, I noticed few things while going through the code. But I need to...

I will remove bpp inferred from pixel format as this doesn't mean much, but for color-depth I would like to still keep it, because it is the target depth that...

To be more explicit, I put it in own label. ![image](https://github.com/mpv-player/mpv/assets/1126053/e0a684cb-d7cd-41b8-b980-5bf7fee0b76e) Let me know if you still think it is useless.