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

Add Pico Passthrough #297

Closed
wants to merge 1 commit into from

Conversation

Rongxuan2024
Copy link

I'm a newbie in XR development, if there is something to be corrected, please help me out XD

@mikeskydev
Copy link
Member

Hi, thanks a lot for the passthrough PR! I'd planned to add it, but I hadn't looked at the relevant API.

As this is a PR to a branch that's not merged yet, would you mind if I took the specific changes we need for passthrough and add them directly to the current PR? I'll make sure the commits are set with you as the author. There's also some changes to the workflow needed as this PR stands, as you'll need to assume that the Pico SDK isn't in the project otherwise there will be compiler errors.

@Rongxuan2024
Copy link
Author

Rongxuan2024 commented Oct 10, 2022

Hi, thanks a lot for the passthrough PR! I'd planned to add it, but I hadn't looked at the relevant API.

As this is a PR to a branch that's not merged yet, would you mind if I took the specific changes we need for passthrough and add them directly to the current PR? I'll make sure the commits are set with you as the author. There's also some changes to the workflow needed as this PR stands, as you'll need to assume that the Pico SDK isn't in the project otherwise there will be compiler errors.

@merulaxr Sure, let's do it. In my code, the passthrough feature is realized with API related to the guardian boundary.
static void EnableSeeThroughManual (bool value)

@Rongxuan2024
Copy link
Author

Hi, thanks a lot for the passthrough PR! I'd planned to add it, but I hadn't looked at the relevant API.

As this is a PR to a branch that's not merged yet, would you mind if I took the specific changes we need for passthrough and add them directly to the current PR? I'll make sure the commits are set with you as the author. There's also some changes to the workflow needed as this PR stands, as you'll need to assume that the Pico SDK isn't in the project otherwise there will be compiler errors.

@merulaxr
Plus, In my attempt, some of the brush shaders didn't work well with the video seethrough stream, probably we can work on it together.

@mikeskydev
Copy link
Member

Great, thanks!

some of the brush shaders didn't work well with the video seethrough stream

Had literally just tested this myself, we had the same issue with Oculus passthrough and I'm trying to remember how it was solved. In Oculus, we set the passthrough to be an underlay layer but Pico doesn't use a Game Object, so I'm not sure how best to change that. It may also not be the solution that I'm remembering.

@Rongxuan2024
Copy link
Author

Probably to change the shader blend mode or texture type of the brushes may help, I can't script shaders, so I'll try the texture later

@mikeskydev
Copy link
Member

Ideally we shouldn't need to change the brushes, textures or shaders as we don't want to change how it looks between apps that use the toolkit, it should be something we tweak that's Pico related!

@Rongxuan2024
Copy link
Author

Absolutely, I agree, keeping the brushes unchanged is good for the project‘s integrity as a whole. Maybe it's on the next update of Pico's VST configuration, hoping it's not too late haha (ToT)

@mikeage
Copy link
Member

mikeage commented Oct 10, 2022

As this is a PR to a branch that's not merged yet, would you mind if I took the specific changes we need for passthrough and add them directly to the current PR? I'll make sure the commits are set with you as the author. There's also some changes to the workflow needed as this PR stands, as you'll need to assume that the Pico SDK isn't in the project otherwise there will be compiler errors.

You can also just change the target....

@mikeskydev mikeskydev added the enhancement Feature added label Dec 5, 2022
@mikeskydev mikeskydev marked this pull request as draft December 5, 2022 15:27
@Rongxuan2024 Rongxuan2024 marked this pull request as ready for review December 8, 2022 15:19
@Zomby2D
Copy link

Zomby2D commented Dec 9, 2022

In my attempt, some of the brush shaders didn't work well with the video seethrough stream

The author of this video compiled your PR and "fixed" the transparent brushes by making the passthrough darker.

@andybak andybak marked this pull request as draft February 24, 2023 12:07
@Rongxuan2024 Rongxuan2024 marked this pull request as ready for review March 7, 2023 14:30
@scottgwald
Copy link

@MuRongxuan1998 I don't have the build environment set up - is there any way you could send me an apk with Pico passthrough enabled? Also, are you working with Pico 3 or Pico 4? Shouldn't make a difference for passthrough, I don't think?!

@scottgwald
Copy link

scottgwald commented Apr 21, 2023

@mikeskydev a kind comment that some people would be very appreciative of getting this merged - I'm eager to use it and I don't have my build tools set up at the moment. I really appreciate what you're doing by maintaining and enhancing this project. It's very powerful for the whole (open) ecosystem. Thank you for your time and effort.

@mikeskydev mikeskydev marked this pull request as draft April 25, 2023 11:05
@mikeskydev
Copy link
Member

@scottgwald per the previous comments, I'm not going to merge an improper fix that compromises the look and feel of the app. We have recently got some leads on the issues facing the brushes though so hope to have them fixed properly soon!

@scottgwald
Copy link

Hope the recent leads will help get it across the finish line!

@Rongxuan2024 Rongxuan2024 marked this pull request as ready for review August 6, 2023 06:53
@Rongxuan2024
Copy link
Author

@MuRongxuan1998 I don't have the build environment set up - is there any way you could send me an apk with Pico passthrough enabled? Also, are you working with Pico 3 or Pico 4? Shouldn't make a difference for passthrough, I don't think?!

Hi, you can get a better version via the link below this video, and PICO 3 & 4 share the same API for passthrough. Sorry for the late reply :)

@andybak andybak marked this pull request as draft February 28, 2024 13:10
@wei-kris
Copy link

this video

It looks good, but do you know what he did to achieve it?

@andybak
Copy link
Contributor

andybak commented Jul 15, 2024

It looks good, but do you know what he did to achieve it?

He implies he just compiled this PR.

@mikeskydev
Copy link
Member

Thanks for the submission, now superseded by the work in #798 directly with OpenXR and composition layers!

@mikeskydev mikeskydev closed this Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature added
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants