PhonologicalCorpusTools / SLPAA

5 stars 0 forks source link

[BUG] Radio button without purpose in Relation module #273

Closed maggieer closed 6 months ago

maggieer commented 7 months ago

Describe the bug The top left corner of the Relation module contains a partially cut-off radio button with no identifying label. It is selectable with no error code.

To Reproduce Steps to reproduce the behavior:

  1. Open SLPAA
  2. Open any corpus/load a new corpus
  3. Enter relation module
  4. See floating radio button at upper left

Expected behavior I expected all radio buttons to have a clear function or at least a label.

Error messages I noticed the following error message the second I started SLP-AA, but it showed up before even choosing to start/load a corpus, so I'm not sure it's related.

C:\Users\maggi\Documents\GitHub\SLPAA\src\main\python\models\movement_models.py:1084: SyntaxWarning: invalid escape sequence '\['
  newlistdisplay = re.sub(" \[.*\]$", "", currentlistdisplay)

Screenshots relation module_01_22

Operating system and SLP-AA version

kvesik commented 7 months ago

Thanks, @maggieer -- The mystery radio button is a known issue (#188) but the error message from the terminal is likely unrelated, so thank you for that!

kvesik commented 7 months ago

@maggieer I don't get the same error on my machine, but I think I know why it's happening. Could you please help me confirm that I've fixed it? Please...

  1. Fetch & pull branch 273 in github desktop (you can just type 273 in the "current branch" search bar):

    image
  2. Run SLP-AA from terminal as usual, and check for that error message. Does it come up still, or is it gone?

  3. Go back to github desktop and switch back to the main branch so you're back to running things as normal (and for fun, let's see if I can fit the word "back" into this sentence a 4th time... yep, I did!).

maggieer commented 7 months ago

Sorry about that @kvesik, I couldn't find the existing issue about the mystery button!

I can confirm the error message disappeared on branch 273.