carrien / free-speech

Analysis and plotting code for speech neuroimaging experiments.
MIT License
4 stars 5 forks source link

check_audapterLPC median point & play audio button #130

Closed cwnaber closed 6 months ago

cwnaber commented 7 months ago

Adds 2 features to the check_audapterLPC gui:

  1. New "Reference point" selectable list which can switch between "mean" (default) and "median"
  2. New "Play" button which plays audio for the selected trial

Other code changes:

  1. Added a second input parameter "params" which controls (1.) mean vs median reference point at startup, and (2.) how the selected trials are picked at startup. For "mean" reference point, it's the trial furthest from the mean. For "median" reference point, it's the trial closest to the median.
  2. Shifted several of the GUI buttons on the right panel to make room for "reference point" section
  3. Previously, selecting a new trial (ie, clicking on a dot) would set the formants to "visible" and vowel bounds to "visible", even if the user had previously toggled them off. Now, the visibility status of formants and vowel bounds is not changed when the user selects a new trial.
  4. The plus sign object (indicating the mean or median point for each vowel) was moved to the bottom of the stack of objects on the figure. This way, the user can select a dot even if it overlaps with a plus sign.

Showing the default behavior, where "mean" is the reference point image

Showing the behavior when "median" is the reference point. And calls out the Play button. The plus sign indicating the median is hidden behind each of the selected points image

parrell commented 7 months ago

It would be good to be able to see the median value even when the median token is selected. Would it be possible to increase the "+" marker size, generally, such that the tails extend past the edge of the circles for each token?

cwnaber commented 7 months ago

Changed the '+' marker size from 6 (default) to 12. Seems OK to me.

image

image

image

cwnaber commented 6 months ago

I verbally got the OK from Carrie to merge