oss-slu / Mouser

GNU General Public License v3.0
1 stars 7 forks source link

Fixes #169 #211

Closed jackbelyeu closed 3 months ago

jackbelyeu commented 3 months ago

fixes #169

What was changed?

Changes are to new_experiment_ui.py, the file responsible for the ui when users are first creating a new experiment.

Why was it changed?

As well as fixing issue#169, these changes also fix a bug. Previously the event handler code was incorrectly implemented and the program checked required fields only when the investigators or measurement fields were edited. Now, the program checks required fields with the previous event handler. I also removed a duplicate method that defined which required fields to check before enabling the next button.

disablednextbutton enablednextbutton

How was it changed?

Changed configuration parameter when button is created.

Added code in init to properly handle binding entry widgets.

*Issue #169 mentions raising a warning when users click 'next' with missing data. Along with these changes, there is also some error handling already implemented with the check_animals_divisible() method **I also added two modules to requirements.txt - playsound, pyaudio. Both were required to run the program locally. (I got OSError when installing playsound. Running pip install --upgrade setuptools wheel fixed it.)