Add documentation about OpenXR spatial entities #11015
Conversation
AThousandShips
added
the
waiting on PR merge
There was a problem hiding this comment.
I noticed a few grammatical issues and typos so I wanted to make a review addressing some of them. I saw that it was also a commonality in this PR to omit commas after dependent clauses, so I left in 2 cases where I noticed it occurred, but I can go back and mark more where I noticed them.
| OpenXR spatial entities | ||
| ======================= | ||
|
|
||
| For any sort of augemented reality application you need to access real world information and be able to |
There was a problem hiding this comment.
Typo, "augemented" should be "augmented."
| It has a very modular design. The core of the API defines how real world entities are structured, | ||
| how they are found and how information about them is stored and accessed. | ||
|
|
||
| Various extensions are added ontop that implement specific systems such as marker tracking, |
| Spatial anchors allow us to map real world location in our virtual world in such a way that the | ||
| XR runtime will keep track of these locations and adjust them as needed. |
There was a problem hiding this comment.
Use of singular "location" in "real world location" but use of plural "locations" in "... will keep track of these locations"
|
|
||
| Spatial anchors allow us to map real world location in our virtual world in such a way that the | ||
| XR runtime will keep track of these locations and adjust them as needed. | ||
| If supported anchors can be made persistent which means the anchors will be recreated in the correct |
There was a problem hiding this comment.
This gets nitpicky since it's been a commonality in this PR to not include commas after dependent clauses but this is my first time noticing it.
Original comment was
Add a comma after "If supported."
| When needed the location of the spatial anchor will be updated automatically, the pose on the | ||
| related tracker will be updated and thus the :ref:`XRAnchor3D<class_XRAnchor3D>` node will | ||
| reposition. |
There was a problem hiding this comment.
This is kinda nitpicky, but I would write it to add a comma after "When needed," and change the comma after "automatically" to a period to prevent this sentence from being a run on.
There was a problem hiding this comment.
Actually, looking at it again, the comma after automatically would probably be better as a semicolon ';' instead of a period.
| related tracker will be updated and thus the :ref:`XRAnchor3D<class_XRAnchor3D>` node will | ||
| reposition. | ||
|
|
||
| When a spatial anchor has been made persistent a Universally Unique Identifier (or UUID) is |
There was a problem hiding this comment.
This gets nitpicky since it's been a commonality in this PR to not include commas after dependent clauses but this is my second(ish) time noticing it.
Original comment was
Comma after persistent as "When a spatial anchor has been made persistent" is a dependent clause.
|
|
||
|
|
||
| Finally we can process our snapshot. Note that we are using :ref:`OpenXRAnchorTracker<class_OpenXRAnchorTracker>` | ||
| as our tracker class as this already has all the support for anchors build in. |
There was a problem hiding this comment.
"build in" should be "built in" since it's past tense.
| With April tags and Aruco markers binary data is encoded based which you again can access when a marker is found, | ||
| however you need to configure the detection with the correct decoding format. | ||
|
|
||
| As an example we'll create a spatial context that will find QR codes adn Aruco markers. |
| Some XR runtimes do require vendor extensions to enable and/or configure this process | ||
| but the data will be exposed through this extension. | ||
|
|
||
| The code we write up above for the spatial manager will already detect our new planes. |
There was a problem hiding this comment.
"write" probably should be "wrote" for past tense.
This PR adds documentation for the spatial entities implementation being added here: godotengine/godot#107391