search

immersive-web / anchors

https://immersive-web.github.io/anchors/
Other
50 stars 20 forks source link

add support for persistentAnchors to XRSession #80

Closed cabanier closed 1 year ago

cabanier commented 1 year ago

Preview | Diff

cabanier commented 1 year ago

rasied from https://github.com/immersive-web/real-world-geometry/issues/37

cabanier commented 1 year ago

@bialpio it seems we forgot to merge this. Can you approve?

cabanier commented 1 year ago

Approving since we mark the new attribute as unstable but we need to start putting some text that describes how this is initialized / when is it cleared up / etc.

Since it reflects the internal map and its behavior is specified, isn't that enough?

bialpio commented 1 year ago

Approving since we mark the new attribute as unstable but we need to start putting some text that describes how this is initialized / when is it cleared up / etc.

Since it reflects the internal map and its behavior is specified, isn't that enough?

IIRC we were chatting about specifying how the map is persisted across sessions (and what is the key by which it's persisted - is it per origin?) and when does it get cleared up (i.e. when clearing up history for the origin)? I don't recall what was the final outcome, but IIUC the way the spec is written right now, nothing will end up being persisted across sessions (I'm assuming that the session's map of persistent anchors always starts empty, because we don't say anywhere that it doesn't).