compas-dev / compas-dev/compas_xr

Reconsider the QR model stuff

Open
#28 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
17
Forks
7
PR merge metrics
No merged PRs in 30d

Description

Same kind of question above applies here. This was a very specific method to support deserialization on the `compas_xr_unity_assembly` [application](https://github.com/compas-dev/compas_xr_unity_assembly/blob/f1516ca568b101447507aebc28a594bdc358df3e/Assets/Scripts/DatabaseManager.cs#L817). _(because it was a bit easier in the application logic to deserialize as an assembly when we were doing all the manual deserialization)_ This is just a general comment. I think with the move to proto this should change kind of specific, but a good discussion point I think....😄

_Originally posted by @jckenny59 in https://github.com/compas-dev/compas_xr/pull/23#discussion_r3829994002_

Contributor guide

Open the contributing guide

Research direction

Start with the linked DatabaseManager.cs location around line 817 and read the originating discussion in pull request #23. Compare that application-specific assembly deserialization with the proposed move to proto; done requires an agreed direction for the QR model and its deserialization approach.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, unity
Domain
backend, data
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.