hackzone / cz20-badge

13 stars 5 forks source link

TTS loop even when having a metadata.json #11

Closed therealprocyon closed 4 years ago

therealprocyon commented 4 years ago

I have a metadata.json set, as well as made my own app, but it becomes a buggy name and the tts keeps looping...

tjclement commented 4 years ago

Do any errors shop up in the terminal during the loop?

therealprocyon commented 4 years ago

[0;37;40mStarting app 'launcher'... Preparing TTS: Animal Crossing TTS: Success Preparing TTS: Animal Crossing Starting app 'launcher'... Starting app 'ACFAQ'... key event i2c read error [Errno 5] EIO key event i2c read error [Errno 5] EIO

tjclement commented 4 years ago

Interesting, so even though the TTS download succeeds, it tries to download it again afterwards. Do you have /flash/cache/appnames/animal_crossing.mp3 in your file system?

therealprocyon commented 4 years ago

No, it's named ACFAQ.mp3

therealprocyon commented 4 years ago

On windows it says "This file isn't playable. That might be because the file type is unsupported, the file extension is incorrect, or the file is corrupt." And it's corrupted

tjclement commented 4 years ago

Yeah downloading mp3 files as a zip in the web interface currently corrupts them somehow. We should log that as a separate issue; perhaps you would be kind enough to do it?

As for the TTS: does /flash/apps/ACFAQ/metadata.json exist? Note that the app "slug" name is case sensitive here

therealprocyon commented 4 years ago

yeah it exists, apparently it generates as .MP3 with caps...

therealprocyon commented 4 years ago

Wait, slug is missing

therealprocyon commented 4 years ago

lemme check again

therealprocyon commented 4 years ago

still nothing

therealprocyon commented 4 years ago

has to do wiith it generating an all-caps filename

therealprocyon commented 4 years ago

it generates all caps and then doesn't work when the slug is in all-caps. So, setting it to acfaq generated acfaq.mp3 instead of ACFAQ.MP3 then it worked

tjclement commented 4 years ago

Cheers! Should be fixed in firmware 2104 (uploading, should be available in ~10 mins). Could you check if it works and close the issue if it does?

therealprocyon commented 4 years ago

sure

therealprocyon commented 4 years ago

fixed, can be closed