search

memphis-iis / mofacts

8 stars 1 forks source link

Audio error for non audio tdf and stims #1457

Closed imrryr closed 8 months ago

imrryr commented 8 months ago

My stim file or tdf is suddenly throwing this error. It all worked for a set of only 10 stims, could a bug in one of the 300 stims I added cause this...

image statsadaptmc.zip

JRustyHaner commented 8 months ago

@imrryr this occurs because stimDisplayTypeMap[currentStimuliSetId] is undefined. JavaScript can't check for hasAudio here, because its parent doesn't exist. I'll take a look more into it.

MegaGeese commented 8 months ago

I've seen this error but it usually only happens shortly after uploading/overwriting the tdf package. Just to confirm did you upload/overwrite the tdf right before this error appeared?

imrryr commented 8 months ago

Yes indeed that is what I did right before.

On Fri, Apr 5, 2024, 11:50 AM August White @.***> wrote:

I've seen this error but it usually only happens shortly after uploading/overwriting the tdf package. Just to confirm did you upload/overwrite the tdf right before this error appeared?

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

imrryr commented 8 months ago

@MegaGeese @JRustyHaner Can I fix this just be changing the filename or something? Or can we deploy if not please? This is MOFACTS2 only at this time

MegaGeese commented 8 months ago

it should fix itself if you refresh the browser

On Apr 5, 2024, at 1:32 PM, Philip Pavlik @.***> wrote:

 CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and trust the content is safe.

@MegaGeesehttps://github.com/MegaGeese @JRustyHanerhttps://github.com/JRustyHaner Can I fix this just be changing the filename or something? Or can we deploy if not please? This is MOFACTS2 only at this time

— Reply to this email directly, view it on GitHubhttps://github.com/memphis-iis/mofacts/issues/1457#issuecomment-2040402890, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMAXGV7EFVGWBNOSWCSDAZDY33U2FAVCNFSM6AAAAABFZK2I2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBQGQYDEOBZGA. You are receiving this because you were mentioned.Message ID: @.***>