Skip to content

Conversation

@BastiaanOlij
Copy link
Member

@BastiaanOlij BastiaanOlij commented Jun 18, 2023

Still working on this one but making a slightly more complex demo.

The demo now includes a spectator view in which we render the players wands (and will also render the players glasses once we fix another issue).

The wands allow you to move around the balls held within the container.

image

Definately can be improved loads by making a nicer container and give the balls some textures so we can see them rolling around (or maybe make a nicer game, I'm thinking air hockey)

@BastiaanOlij BastiaanOlij added the enhancement New feature or request label Jun 18, 2023
@BastiaanOlij BastiaanOlij added this to the 1.0.0 milestone Jun 18, 2023
@BastiaanOlij BastiaanOlij self-assigned this Jun 18, 2023
@patrickdown
Copy link
Collaborator

This looks cool. My only comment so far is maybe instead of hard coded paths to the viewport and origin node in the Glasses scene those nodes should be searched for? I don't know how variable a custom glasses scene might get.

@BastiaanOlij
Copy link
Member Author

This looks cool. My only comment so far is maybe instead of hard coded paths to the viewport and origin node in the Glasses scene those nodes should be searched for? I don't know how variable a custom glasses scene might get.

That is in part why we have the base scene to be extended, that guarantees certain paths. That said, I think this can be improved upon.

@BastiaanOlij BastiaanOlij marked this pull request as ready for review June 21, 2023 08:49
@BastiaanOlij
Copy link
Member Author

BastiaanOlij commented Jun 21, 2023

@patrickdown I've added that you can use the wand as a pointer to pick up balls (use trigger) and added the ability to move around with the joystick.

I'm not sure if this is the best type of example but it is showing a few things off. I need to fix some other things before I can add more.

I think this is worth merging as is and just keep building upon, Maybe add other things around that we can interact with.

@BastiaanOlij BastiaanOlij marked this pull request as draft November 12, 2023 10:46
@BastiaanOlij
Copy link
Member Author

Just moved this back to draft, I think we need something much nicer for our start. The work Malcolm is doing will help loads.

@BastiaanOlij
Copy link
Member Author

I'm closing this pull request, it's outdated and we have better demos now as part of the toolkit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants