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

Reading Mode Doesn't work with URP #10337

Closed
pajama opened this issue Nov 22, 2021 · 9 comments
Closed

Reading Mode Doesn't work with URP #10337

pajama opened this issue Nov 22, 2021 · 9 comments
Assignees

Comments

@pajama
Copy link

pajama commented Nov 22, 2021

Reading Mode Doesn't work with URP. Likely because OnPreCall is never called. RenderPipelineManager events needs to be implemented.

Also RenderViewportScale doesn't work with URP, but I'm guessing this is an underlying Unity OpenXR issue?

To reproduce

Switch to URP. Try the ReadingModeDemo.

Expected behavior

Reading mode demo should perform as expected.

  • Unity Version 2020.3.20f1
  • MRTK Version 2.7.2
  • HoloLens 2
    -OpenXR
@pajama pajama added the Bug label Nov 22, 2021
@polar-kev
Copy link
Contributor

Thanks for filing @pajama

@polar-kev polar-kev added the URP label Nov 24, 2021
@polar-kev
Copy link
Contributor

FYI @KevinKennedy.

@Zee2
Copy link
Contributor

Zee2 commented Aug 31, 2022

@Cameron-Micka , is this something we're thinking about for MRGT?

@Cameron-Micka
Copy link
Member

Oh, what is the "reading mode" demo?

@keveleigh
Copy link
Contributor

Info on reading mode: https://docs.microsoft.com/hololens/hololens2-display#what-improvements-are-coming-that-will-improve-hololens-2-image-quality

Some MRTK2 background: #8344, #8651, #8808

@Cameron-Micka
Copy link
Member

Thanks @keveleigh, I had no idea this was a feature. This is not something we've considered for MRGT but could be something we think about. We already have defines in place to conditionally compile URP specific logic so should be too hard to support both pipelines.

I'm open to tracking this issue here: https://github.com/microsoft/MixedReality-GraphicsTools-Unity/issues or if someone has time to make the change, I'm happy to review the PR. 😊

@Zee2
Copy link
Contributor

Zee2 commented Aug 31, 2022

@Cameron-Micka, created an issue for ya :)

@Cameron-Micka
Copy link
Member

Thanks!

@Zee2
Copy link
Contributor

Zee2 commented Sep 2, 2022

Closing as duplicate of the new MRGT issue (microsoft/MixedReality-GraphicsTools-Unity#97)

@Zee2 Zee2 closed this as not planned Won't fix, can't repro, duplicate, stale Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants