Left laser position is missing in sample.csv

med-material / Whack_A_Mole_VR

Whack-A-Mole in VR

MIT License

3 stars 15 forks source link

Left laser position is missing in sample.csv #245

Closed aldsanms closed 1 year ago

aldsanms commented 1 year ago

@Xav1204

The following columns are missing from the sample.csv file:

LeftControllerLaserPosWorldX
LeftControllerLaserPosWorldY
LeftControllerLaserPosWorldZ
I need it for my visualization.

Xav1204 commented 1 year ago

@aldsanms you can test with the following files. Tell me if it's not right. CSV.zip

aldsanms commented 1 year ago

@aldsanms you can test with the following files. Tell me if it's not right. CSV.zip

Hey,

I'm fixing some bugs in my program, but I have a few questions:

is "DistractorLeft Mole Expired" and "DistractorLeft Mole Spawned" in the event column the same as "Fake Mole Expired" and "Fake Mole Spawned" on your previous version?
Why "left" on "DistractorLeft Mole..."? I never see "right"

aldsanms commented 1 year ago

@Xav1204

I have an issue in your last CSV, in "ControllerName" column of event.csv file, the value is still null. I need it to know if I should show motion or not.

Xav1204 commented 1 year ago

@aldsanms Yes, Distractor Mole Spawned/Expired are the same as Fake mole in the previous version. We have Distractor Left and Right, it's probably that no Distractor Right spawned during this session. For the controller name, it appears when you hit a mole to specify with which controller you hit the mole.

bastianilso commented 1 year ago

@aldsanms in the Event column, there is the "Controller Main Set Both" field. Maybe looking for this value is a better indicator of which controllers are currently active.

@Xav1204 maybe we should emit a "Controller Main Set Right" on startup of Whack-A-Mole, since this is the default value?

aldsanms commented 1 year ago

@Xav1204

Can you send me the CSV of a more complete game with head movements and mole interaction?

Xav1204 commented 1 year ago

@aldsanms you can try with these files to see if it's right. CSV.zip

aldsanms commented 1 year ago

@aldsanms you can try with these files to see if it's right. CSV.zip

Everything works fine with these files. thank you.

aldsanms commented 1 year ago

I think this issue can be closed.

bastianilso commented 1 year ago

Controller state along with other modifiers are now announced with every game start as per commit 11924a120d637124de46354dba577c29faaa0638