med-material / ArduinoReactionSynchTests

Collection of Arduino Scripts for measuring human reaction time and synchronization abilities.
MIT License
0 stars 1 forks source link

Participant ID defaulting back to 1 #13

Closed Marsfeldt closed 2 years ago

Marsfeldt commented 2 years ago

I change the participant ID in the logger to e.g. 2, but all the data is saved under ID 1.

Enter this: image

Gets this (but with all the new data): image

Recreated on two computers.

bastianilso commented 2 years ago

Hey Milo, I can't reproduce your issue - changing PID from 1 to 2 works fine here and also appears on the server. And I see a few students have already used other PIDs than "1" (e.g. 40).

image

Did the PIDs show up correctly when you saved to CSV, Milo?

mathiassand commented 2 years ago

I can reproduce the same error on mac. In addition, when I create a new PID (e.g. 31), it's saved in the .csv file as PID 2, which is the first PID I made.

bastianilso commented 2 years ago

OK so the issue is that the Reconnect button is not updating the logged PID properly. so the back button is not updating it. But restarting the arduino logger works fine for updating PIDs.

bastianilso commented 2 years ago

fixed in b4fd8ebf64a73df4cd754de0e6c7cb18074b5e66