Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Scene Understanding demo is dependent on Spatial Awareness demo #9431

Closed
MaxWang-MS opened this issue Mar 4, 2021 · 1 comment · Fixed by #9479
Closed

Scene Understanding demo is dependent on Spatial Awareness demo #9431

MaxWang-MS opened this issue Mar 4, 2021 · 1 comment · Fixed by #9479
Assignees
@MaxWang-MS
Labels
Bug
Projects
[Retired] MRTK Backlog
Milestone
MRTK 2.6.1

Comments

@MaxWang-MS
Copy link
Contributor

Describe the bug

In MRTK 2.6, a script in the Scene Understanding demo is referencing another script in the Spatial Awareness demo, which means when MRTK is consumed via UPM/Mixed Reality Feature Tool the Spatial Awareness sample in the examples package must be imported before importing the experimental Scene Understanding sample or otherwise there will be compiler errors in the console. This issue does not affect people using MRTK via unitypackages.
@MaxWang-MS MaxWang-MS added the Bug label Mar 4, 2021
@david-c-kline david-c-kline added this to the MRTK 2.6.1 milestone Mar 5, 2021
@keveleigh
Copy link
Contributor

@MaxWang-MS Should we move this to 2.6.2 / 2.7?

@keveleigh keveleigh added this to Needs Triage in [Retired] MRTK Backlog via automation Mar 10, 2021
@polar-kev polar-kev moved this from Needs Triage to To do in [Retired] MRTK Backlog Mar 10, 2021
@polar-kev polar-kev moved this from To do to In progress in [Retired] MRTK Backlog Mar 10, 2021
@MaxWang-MS MaxWang-MS mentioned this issue Mar 10, 2021
Merged
@MaxWang-MS MaxWang-MS linked a pull request Mar 10, 2021 that will close this issue
Decouple the scene understanding sample from the spatial awareness one #9479
Merged
[Retired] MRTK Backlog automation moved this from In progress to Done Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MaxWang-MS MaxWang-MS

Labels
Bug
Projects
[Retired] MRTK Backlog
Done - Archive
Milestone
MRTK 2.6.1
3 participants
@keveleigh @david-c-kline @MaxWang-MS