med-material / VREyeTrackingAccuracyTest

VR Tool to Test the Accuracy of the Pupil Labs Eye Tracking Devices (Unity 2018.2.7f1)
MIT License
2 stars 4 forks source link

MainMenu UI: Connect the Dropdown to the actual code #13

Closed bastianilso closed 5 years ago

bastianilso commented 5 years ago

When we implement and revamp the main menu, we should retain the functionality which the old main menu provided. Therefore the code needs to be updated to work using the new main menu dropdown design.

YoNeXia commented 5 years ago

To connect the Dropdown from the new UI, I am going to check the old version of the program. I run it in Unity with the old UI and check how its work. When I find it, I am going to implement new code for using the new dropdown.

YoNeXia commented 5 years ago

I implement the dropdown inside the code. Before, the code use just a if else to set up the time for switch circles. I replace it by a switch, but this one does not work... I am trying to understand why.

Code

targetLifeSpan is a public static float

YoNeXia commented 5 years ago

I set a fixed value into the old and the new program... Fixed value The old program work but not the new, so I am trying to find a solution, but I am blocked for the moment.

bastianilso commented 5 years ago

@YoNeXia how does targetLifeSpan work? Where is it used?

bastianilso commented 5 years ago

@YoNeXia what can we do to narrow down the possible sources for this issue? What have you tried so far?

YoNeXia commented 5 years ago

@bastianilso The targetLifeSpan variable is a float. In old program, it takes the value of the apparence time of circles in the test. It used in StartConfig.cs, and some operation are made in CircleLife.cs. And I connect the dropdown with switch case as you can see upper. As my switch case does not work, I fix the value, and as I said before he does not work in new program. I am trying to find a solution now, but no idea for the moment... And I do not have any idea where is the source of this issue, I would to find it.

YoNeXia commented 5 years ago

@bastianilso Ok I found it. Its because we fix default values at old toggles. And the old toggle inputMode in StartConfig.cs control the mode of the circles, automatic with timer or manual with spacebar. 😅 I am going to resolve it now. 👍

bastianilso commented 5 years ago

@YoNeXia Your Pull Request #16 has been merged, so I am closing this issue.