search

InteractiveMechanics / nmai-nk360-interactives

0 stars 0 forks source link

[x] Audio not working on annotators #624

Closed amberreeves closed 6 years ago

amberreeves commented 6 years ago

Laurie has reported that the audio isn't working on our instances of the annotator or any of the new instances that they have created. Can we look into reasons/solutions for this?

jmaj commented 6 years ago

What's the URL on NMAI's server?

mtedeschi commented 6 years ago

You have to log in via Citrix since they're private URLs. I can take a look at it.

amberreeves commented 6 years ago

@mtedeschi laurie confirmed that the audio on her device is working and on. she also sent these examples.

Example created by IM http://nmai-webdev01.si.edu:84/nk360/pnw-history-culture-regions/pnw1-markup1/

Example created by NMAI http://nmai-webdev01.si.edu:84/nk360/pnw-history-culture-regions/pnw1-markup2/

mtedeschi commented 6 years ago

It's a path issue with the audio files—need to be absolute, not relative.

Right now, they are: "../shared/lib/im/audio/PL_Crow.mp3"

And should be: "/nk360/shared/lib/im/audio/PL_Crow.mp3"

I can adjust this in all the annotators we have, and they'll need to follow that in the future, too.

amberreeves commented 6 years ago

ok, i let laurie and david know about this.

amberreeves commented 6 years ago

@mtedeschi can you field this question from david?

Regarding /nk360/shared/lib/im/audio/PL_Crow.mp3

I am confused. If this is in shared folder, IM can use the absolute path. However, PL_Crow.mp3 is a tribe-specific content. Why did they put in the shared folder? As I said, IM only needs put shared core code in /shared/lib/im

mtedeschi commented 6 years ago

I'm not going to address David's concern. Our assumption was that audio clips could be used across different modules (and would have been used beyond just Annotators), so we made them shared to not duplicate audio files across each interactive/activity.

amberreeves commented 6 years ago

i passed this onto david and this was his response:

Okay, please update so that an absolute path is used. We created two annotators yesterday. Once the code is updated, will we need to create them again?