regzo2 / PicoStreamingAssistantFTUDP

Demonstration of how to fetch face tracking data via the Pico Streaming Assistant.
27 stars 11 forks source link

1.4 stuck on initializing #20

Closed buddybopper closed 6 months ago

buddybopper commented 6 months ago

The newest module (1.4) causes vrcft to be stuck on initializing. I used the one from the module registry, not the github download, idk if that was the problem or not.

rogermiranda1000 commented 6 months ago

Can you show the logs? Also, is PICO Connect and SteamVR launched?

buddybopper commented 6 months ago

Idk how to find the logs, pico streaming assistant is running and I am using vrc through steamvr

rogermiranda1000 commented 6 months ago

VRCFT has a logs tab, check what does it say. It seems like it works for some users, so I don't think it's a problem with the .dll...

buddybopper commented 6 months ago

latest.log okok found it

TerayTech commented 6 months ago

same problem with 1.4, and VRCFT version 5.2.4.0 roll back to 1.3 solve the problem, and everytime reopen VRCFT it will update automaticlly and you have to re-install 1.3 manually

TerayTech commented 6 months ago

latest.log and here's my log, my operations are bellow: open pico stream assistant -> open steamVR -> open VRCFT it doesn't affect whether if VRC is on or off

rogermiranda1000 commented 6 months ago

@buddybopper @TerayTech you say on 1.3 didn't happend, can you share the logs on that case? Thanks, and sorry for the inconvenience.

rogermiranda1000 commented 6 months ago

Also, can you try to launch VRCFT once SteamVR is on? Not before Edit: I just saw you already launched SteamVR

buddybopper commented 6 months ago

latest.log here's the 1.3 log

rogermiranda1000 commented 6 months ago

I noticed how you both use Streaming Assistant (and I didn't check that...) maybe that's the issue? I'll check it

rogermiranda1000 commented 6 months ago

The issue seems to be on the retry loop, on the Initialize function: https://github.com/regzo2/PicoStreamingAssistantFTUDP/blob/27a199daf055e5a5728625831fb504e8919a3a69/PicoStreamingAssistantFTUDP/Pico4SAFTExtTrackingModule.cs#L69

I'll have to consider the consequences of removing that, and then push an update.

rogermiranda1000 commented 6 months ago

Seems like the while is not needed, as at this part of the code the socket didn't have a timeout yet. If you close VRCFT it will close even if it's still connecting.

However, I noticed another bug: https://github.com/regzo2/PicoStreamingAssistantFTUDP/issues/22

rogermiranda1000 commented 6 months ago

Solved on 1.4.1