search

memphis-iis / mofacts

8 stars 1 forks source link

Button trials not working #1496

Closed imrryr closed 3 months ago

imrryr commented 4 months ago

Can't figure it out. the button trials here TDFpp.zip are doing this:

image

imrryr commented 4 months ago

Here is the actual example that produced the above. I was trying many configurations of the json, since I though this worked since I think it works for my experiment.... TDF11.zip

imrryr commented 4 months ago

This one also failed... TDFpp.zip

imrryr commented 4 months ago

What it appears to be doing is making a single button for every single character in the incorrect answers

MegaGeese commented 4 months ago

I think I have a fix for this on staging and can make a quick pr for this

imrryr commented 4 months ago

No it can't be on staging this is a problem on production. I can't test all of the other stuff that's on staging I just need this fixed.

On Tue, Jun 18, 2024, 9:06 AM August White @.***> wrote:

I think I have a fix for this on staging and can make a quick pr for this

— Reply to this email directly, view it on GitHub https://github.com/memphis-iis/mofacts/issues/1496#issuecomment-2176194022, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADDLPK43D4K454DGXRJD3D3ZIA5FPAVCNFSM6AAAAABJQAEYZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZWGE4TIMBSGI . You are receiving this because you were assigned.Message ID: @.***>

imrryr commented 4 months ago

Okay but this is a problem on production and I don't want to start working on staging for this work that's immediate. Can I talk to you on the phone?

On Tue, Jun 18, 2024, 9:45 AM Philip Pavlik @.***> wrote:

No it can't be on staging this is a problem on production. I can't test all of the other stuff that's on staging I just need this fixed.

On Tue, Jun 18, 2024, 9:06 AM August White @.***> wrote:

I think I have a fix for this on staging and can make a quick pr for this

— Reply to this email directly, view it on GitHub https://github.com/memphis-iis/mofacts/issues/1496#issuecomment-2176194022, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADDLPK43D4K454DGXRJD3D3ZIA5FPAVCNFSM6AAAAABJQAEYZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZWGE4TIMBSGI . You are receiving this because you were assigned.Message ID: @.***>

imrryr commented 4 months ago

Or zoom

On Tue, Jun 18, 2024, 9:46 AM Philip Pavlik @.***> wrote:

Okay but this is a problem on production and I don't want to start working on staging for this work that's immediate. Can I talk to you on the phone?

On Tue, Jun 18, 2024, 9:45 AM Philip Pavlik @.***> wrote:

No it can't be on staging this is a problem on production. I can't test all of the other stuff that's on staging I just need this fixed.

On Tue, Jun 18, 2024, 9:06 AM August White @.***> wrote:

I think I have a fix for this on staging and can make a quick pr for this

— Reply to this email directly, view it on GitHub https://github.com/memphis-iis/mofacts/issues/1496#issuecomment-2176194022, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADDLPK43D4K454DGXRJD3D3ZIA5FPAVCNFSM6AAAAABJQAEYZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZWGE4TIMBSGI . You are receiving this because you were assigned.Message ID: @.***>

MegaGeese commented 4 months ago

Sorry if i wasnt clear, what I meant was that I have already fixed the issue on staging and can copy the fix over for production. There is currently a pull request open right now for this issue that will fix it on produciton #1498

imrryr commented 4 months ago

ahhh... ok, but i still would like to discuss it. I am on teams chat now if you can