Update README to remove OpenVR tests and add links - #61
Conversation
Removed OpenVR integration tests section and added links to video demonstrations.
There was a problem hiding this comment.
Pull request overview
This PR simplifies the README by removing the OpenVR Integration Tests section's table structure and replacing it with three direct video asset URLs.
Key changes:
- Removed the markdown table structure that displayed OpenVR test videos
- Replaced with three plain GitHub asset URLs
- Removed descriptive captions and explanatory notes about the VR features
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| https://github.com/user-attachments/assets/ea81822c-4639-4dac-a033-86cfc7f25acf | ||
|
|
||
| | | | | ||
| |:-------------------------:|:-------------------------:| | ||
| |  |  | | ||
| | *VR Camera Tracking & Rendering* | *Motion Controller Integration* | | ||
| https://github.com/user-attachments/assets/7896ba35-49c9-4a9c-bea1-635bbeab81c8 | ||
|
|
||
| > 🎥 **Note**: Video demonstrations showcasing experimental VR features with the TKD Engine | ||
| https://github.com/user-attachments/assets/f25013cf-f10b-424c-8df6-e827d484072e |
There was a problem hiding this comment.
The URLs are not properly formatted as markdown. In markdown documents, bare URLs should either be wrapped in angle brackets to make them clickable or embedded in markdown syntax. Currently, these URLs will appear as plain text.
Consider using one of these formats:
- Angle brackets:
<https://github.com/user-attachments/assets/...> - Markdown link syntax:
[Description](https://github.com/user-attachments/assets/...) - Markdown embedded video:
Since these appear to be video assets (based on the removed content context), using the image/video markdown syntax would be most appropriate.
| https://github.com/user-attachments/assets/ea81822c-4639-4dac-a033-86cfc7f25acf | ||
|
|
||
| | | | | ||
| |:-------------------------:|:-------------------------:| | ||
| |  |  | | ||
| | *VR Camera Tracking & Rendering* | *Motion Controller Integration* | | ||
| https://github.com/user-attachments/assets/7896ba35-49c9-4a9c-bea1-635bbeab81c8 | ||
|
|
||
| > 🎥 **Note**: Video demonstrations showcasing experimental VR features with the TKD Engine | ||
| https://github.com/user-attachments/assets/f25013cf-f10b-424c-8df6-e827d484072e |
There was a problem hiding this comment.
The removal of descriptive context ("VR Camera Tracking & Rendering", "Motion Controller Integration") and explanatory note makes it unclear what these videos demonstrate. Without captions or descriptions, readers won't know what they're looking at or why these videos are included.
Consider adding descriptive text or captions for each video link to maintain documentation clarity and help users understand what each video demonstrates.
Removed OpenVR integration tests section and added links to video demonstrations.