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

fix(server) HEVC black screen on some linux config #2203

Merged
merged 1 commit into from
Jun 28, 2024

Conversation

Nibor62
Copy link
Contributor

@Nibor62 Nibor62 commented Jun 20, 2024

Fix for #2202

After a bit of bisection, I found out that reverting this line fixed the blackscreen issue on my config.

WARN: even though I found that it fixes the issue I do not precisely understand why it does so.

@Nibor62
Copy link
Contributor Author

Nibor62 commented Jun 20, 2024

So, from what I understand, the gop_frame is the frequency of intra_frame sending. I guess we have a very large number as it is not very usefull in the context of ALVR.
However it might be advised to ask @galister if reverting this modif is ok

@Vixea
Copy link
Collaborator

Vixea commented Jun 20, 2024

May I ask what distro?

@Nibor62
Copy link
Contributor Author

Nibor62 commented Jun 20, 2024

I am on arch

@The-personified-devil
Copy link
Collaborator

Tbh I don't understand why it's that ridiculously high anyways

@Vixea
Copy link
Collaborator

Vixea commented Jun 20, 2024

Because we want to insert idrs/I frames when we want to not at the end of gop so higher sized gop means less "unexpected" idr/I frames

@The-personified-devil
Copy link
Collaborator

Ik, but int max is still ridiculous

@The-personified-devil
Copy link
Collaborator

Would this insert I frames or IDR frames? Because if it's I frames I see no issue with possibly wrongly inserting one every ~4 minutes at 120 fps

@Nibor62
Copy link
Contributor Author

Nibor62 commented Jun 24, 2024

May I ask if you have any news here ?

Newbie question : you that "I" frames were ok, so I guess that IDR are not. What is the different between the two and how do they impact with alvr ?

@The-personified-devil
Copy link
Collaborator

I frames are frames that aren't encoded as differences to previous frames. IDR frames do this but also don't allow any frames before the I frame to be referenced. This will cause pixelated frames because without all the encoding tricks only a pixelated frame can satisfy the bitrate requirements for an IDR frame.

@The-personified-devil
Copy link
Collaborator

Make it conditional for HEVC and it's fine. Weird workaround is way better than not working at all

@zmerp
Copy link
Member

zmerp commented Jun 26, 2024

I would add, the workaround should be documented with a comment explaining the reasons behind it.

@Nibor62
Copy link
Contributor Author

Nibor62 commented Jun 28, 2024

updated

@zmerp
Copy link
Member

zmerp commented Jun 28, 2024

Needs rebasing

@Vixea Vixea merged commit 07d77ff into alvr-org:master Jun 28, 2024
failboat78 pushed a commit to failboat78/ALVR that referenced this pull request Jun 29, 2024
@The-personified-devil The-personified-devil linked an issue Jun 29, 2024 that may be closed by this pull request
5 tasks
ShootingKing-AM pushed a commit to PhoneVR-Developers/ALVR-phonevr that referenced this pull request Jul 5, 2024
* Increase maximum resolution from 4096 to 8192

* decimal instead of hex

* chore: add formatting style for c++

* chore: reformat c++ to new style

* chore: add c++ autoformatter

* chore: Add missed/bugfixed format changes

* fix: routing assumption basis of microphone loopback detection (alvr-org#2209)

* fix(server) HEVC black screen on some linux config (alvr-org#2203)

* refactor(xtask): ♻️ Fix clippy and refactor (alvr-org#2213)

* fix(client_core): 🐛 Fix USB mode (alvr-org#2214)

* refactor(client): ♻️ Remove separate Pico Neo 3 Link platform

* client gfx refactor (7): partially port lobby

* feat(client): ✨ Show hands in lobby

* feat(client_openxr): ✨ Make lobby use max swapchain res (alvr-org#2220)

* feat(client_core): ✨ Thicken lobby grid lines

* fix(client): 🐛 Fix performance regressions on Quest 1

---------

Co-authored-by: shinyquagsire23 <mtinc2@gmail.com>
Co-authored-by: Leonhard Saam <leonhard.saam@yahoo.com>
Co-authored-by: 瑄 <shanboxuan@gmail.com>
Co-authored-by: Nibor62 <Nibor62@users.noreply.github.com>
Co-authored-by: zarik5 <riccardo.zaglia5@gmail.com>
failboat78 pushed a commit to failboat78/ALVR that referenced this pull request Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HEVC black screen on linux
4 participants