LocalJoost / BlogComments

This repo is solely used for comments on https://localjoost.github.io/
4 stars 0 forks source link

Comments on "MRTK2 to MRTK3 - migrating Extension Services to the Reality Collective Service Framework" #420

Open LocalJoost opened 2 years ago

LocalJoost commented 2 years ago

Original article: https://localjoost.github.io/MRTK2-to-MRTK3-migrating-Extension-Services-to-the-Reality-Collective-Service-Framework/

FS178 commented 1 year ago

Hello, I'm trying to run the QRCodeService-MRTK3 project in Unity 2021.2.16. However, it seems that with the version updates, the type IServiceDataProvider is no longer present in the QRCodeTrackingServiceProfile script, which inherits from BaseServiceProfile. I've attempted to make replacements, but unfortunately, although I can build the project, I encounter errors when uploading it to the HoloLens. Is there an alternative solution available? The version of RealityCollective.Utilities is 1.04, and the version of RealityCollective.Service-Framework is 1.02.

LocalJoost commented 1 year ago

Hi @FS178, I am aware of your issue. You see, the architect of the Service Framework, a good friend of mine, is suffering from something almost all software architects - including myself - suffer: a light case of what I call Architecteritis, that is, the tendency to muck around with names, name spaces or other minor things for reasons that make no sense to anybody but themselves - and break the API. To be really honest, when I wrote about it, is was still a preview version. And he just assured me this would never happen again. :D

Anyhew, I have updated the repo to reflect the changes, and updated the repo to the very latest version of the MRTK3, so if you pull this branch, you will probably have a fresh working version out of the box https://github.com/LocalJoost/QRCodeService/tree/MRTK3

FS178 commented 1 year ago

Thank you for your response, and I apologize for the delay in getting back to you. I have retested the uploaded project and successfully used it on HoloLens 2 using Unity 2020.3.35. However, when I tried to port it to the project using Unity 2021.2.16, I didn't encounter any errors during the Unity editor or packaging process. But when I uploaded the packaged project to the HoloLens 2 using Visual Studio, an error occurred. It seems that a variable has been optimized out. I tried to resolve this issue by optimizing and adjusting the editor's optimization settings, but it took a lot of time and did not solve the problem. I have attached a screenshot of the error. That's also the reason for my late response. I believe this could be an issue related to the Unity version.

执友者 @.***

 

------------------ 原始邮件 ------------------ 发件人: "LocalJoost/BlogComments" @.>; 发送时间: 2023年5月17日(星期三) 下午5:47 @.>; @.**@.>; 主题: Re: [LocalJoost/BlogComments] Comments on "MRTK2 to MRTK3 - migrating Extension Services to the Reality Collective Service Framework" (Issue #420)

Hi @FS178, I am aware of your issue. You see, the architect of the Service Framework, a good friend of mine, is suffering from something almost all software architects - including myself - suffer: a light case of what I call Architecteritis, that is, the tendency to muck around with names, name spaces or other minor things for reasons that make no sense to anybody but to themselves - and break the API. To be really honest, when I wrote about it, is was still a preview version.

Anyhew, I have updated the repo to reflect the changes, and updated the repo to the very latest version of the MRTK3, so if you pull this branch, you will probably have a fresh working version out of the box https://github.com/LocalJoost/QRCodeService/tree/MRTK3

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

LocalJoost commented 1 year ago

@FS178 I don't see a screenshot here. I just see few Chinese characters that apparently means "friends". But yeah - if you try random Unity versions, you might run into issues indeed

FS178 commented 1 year ago

void RuntimeType::GetGUID(ll2CppReflectionTypetype,ll2CppArrayguid result)

{

IL2CPP ASSERT(vm::Array:GetLength(guid result) == sizeof(l2CppGuid);

if (type == NULL)

return;

Il2CppClass*klass=vm::Class:Fromll2CppType(type->type):

if (klass >interopData != nullptr)

{

uint8 t*guid=il2cpp_array_addr_with_size(guid_result,1,0);

memcpy(guid, klass->interopData->quid, sizeof(ll2CppGuid):

`       }

}

Error:Exceplion thrown at 0x00007FF807C78A04 (GarneAssembly.dll)in QRCodeService.exe: 0xCC000005: Access violation readinglacatian 0x0000000000000000

I'm glad you took the time to reply to me. The screenshot may not have been sent because the email client doesn't support it. I really hope that even if there are changes in Unity versions, any problems with Unity will be exposed directly in the editor rather than dampening my expectations when I try to upload after packaging. In addition to the errors mentioned above, the VS in the watch window also told me that the "type" variable had been optimized and couldn't be used. I will continue to research and try to find a solution to this problem. After all, now we have the convenient tool of ChatGpt, although sometimes it can mislead us.

执友者 @.***

 

------------------ 原始邮件 ------------------ 发件人: "LocalJoost/BlogComments" @.>; 发送时间: 2023年5月18日(星期四) 晚上8:09 @.>; @.**@.>; 主题: Re: [LocalJoost/BlogComments] Comments on "MRTK2 to MRTK3 - migrating Extension Services to the Reality Collective Service Framework" (Issue #420)

@FS178 I don't see a screenshot here. I just see few Chinese characters that apparently means "friends". But yeah - if you try random Unity versions, you might run into issues indeed

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

FS178 commented 1 year ago

void RuntimeType::GetGUID(ll2CppReflectionTypetype,ll2CppArrayguid result)

{

IL2CPP ASSERT(vm::Array:GetLength(guid result) == sizeof(l2CppGuid);

if (type == NULL)

return;

Il2CppClass*klass=vm::Class:Fromll2CppType(type->type):

if (klass >interopData != nullptr)

{

uint8 t*guid=il2cpp_array_addr_with_size(guid_result,1,0);

memcpy(guid, klass->interopData->quid, sizeof(ll2CppGuid)://The code line that generates the error.

`       }

}

Error:Exceplion thrown at 0x00007FF807C78A04 (GarneAssembly.dll)in QRCodeService.exe: 0xCC000005: Access violation readinglacatian 0x0000000000000000

I'm glad you took the time to reply to me. The screenshot may not have been sent because the email client doesn't support it. I really hope that even if there are changes in Unity versions, any problems with Unity will be exposed directly in the editor rather than dampening my expectations when I try to upload after packaging. In addition to the errors mentioned above, the VS in the watch window also told me that the "type" variable had been optimized and couldn't be used. I will continue to research and try to find a solution to this problem. After all, now we have the convenient tool of ChatGpt, although sometimes it can mislead us.

执友者 @.***

 

------------------ 原始邮件 ------------------ 发件人: "LocalJoost/BlogComments" @.>; 发送时间: 2023年5月18日(星期四) 晚上8:09 @.>; @.**@.>; 主题: Re: [LocalJoost/BlogComments] Comments on "MRTK2 to MRTK3 - migrating Extension Services to the Reality Collective Service Framework" (Issue #420)

@FS178 I don't see a screenshot here. I just see few Chinese characters that apparently means "friends". But yeah - if you try random Unity versions, you might run into issues indeed

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>