search

BIDMCDigitalPsychiatry / LAMP-platform

The LAMP Platform (issues and documentation).
https://docs.lamp.digital/
Other
13 stars 10 forks source link

Spin the Wheel Task NaN Values #773

Closed carlan1 closed 1 year ago

carlan1 commented 1 year ago

It appears that sometimes the Iowa Gambling Task / Spin the wheel task produces NaN values for the amount of money remaining. Please see the attached screenshot:

Screen Shot 2023-08-07 at 10 21 29 AM

Here are the settings associated with the game:

Screen Shot 2023-08-07 at 10 23 44 AM

The researcher ID where the game was assigned is arbra7fwgbzx4ba60nh8 and the activity ID is kp9cnnkgb417ymm4r0jc

Additionally, please add a setting to the game that allows researchers to specify the starting balance.

evinkalengaden commented 1 year ago

We have tested the Spin the wheel game in the dashboard staging using the customized settings as mentioned. Now its working fine .

carlan1 commented 1 year ago

For each wheel (gain and loss wheels) the participant can gain/lose x dollars OR gain/lose 0 dollars. However there is currently no check to make sure that probability(x) + probability(0) = 1. What happens if I choose 50 and 25 for the "probability" options? Will the true probability be 50/(25+50) and 25/(25+50)?

carlan1 commented 1 year ago

Additionally, please add a setting to the game that allows researchers to specify the starting balance.

I do not see this setting yet @evinkalengaden

sarithapillai8 commented 1 year ago

@carlan1

please add a setting to the game that allows researchers to specify the starting balance.

We missed this, sorry. We can update it today. We can check the probability.

sarithapillai8 commented 1 year ago

For each wheel (gain and loss wheels) the participant can gain/lose x dollars OR gain/lose 0 dollars. However there is currently no check to make sure that probability(x) + probability(0) = 1. What happens if I choose 50 and 25 for the "probability" options? Will the true probability be 50/(25+50) and 25/(25+50)?

If probablility 50 and 25 is selected now it works with probablility 50/(25+50) and 25/(25+50). No checking for probability(x) + probability(0) = 1. Please refer the code: https://github.com/BIDMCDigitalPsychiatry/LAMP-activities/blob/master/Spin_Wheel/src/containers/Layout.tsx#L68

@carlan1

sarithapillai8 commented 1 year ago

@carlan1 Balance added for game settings. Do we need top add any conditions for this parameter?

carlan1 commented 1 year ago

Looks good in staging