Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

List index out of range #2981

Closed
Arndorferd opened this issue Aug 20, 2021 · 12 comments
Closed

List index out of range #2981

Arndorferd opened this issue Aug 20, 2021 · 12 comments

Comments

@Arndorferd
Copy link

Describe the bug
Error code and stops listen to further commands

To Reproduce
Steps to reproduce the behavior:
Ask it something, last was listen to mpr in record was supposed to be listen to npr.
It froze and wouldnt listen to other commands
Gave error code

Expected behavior
Should handle the exception and tell me it doesnt understand

Log files
23:27:28.533 | ERROR | 738 | mycroft.client.speech.listener:transcribe:239 | list index out of range

  ^--- NEWEST ---^ 

**Environment (please complete the following information):**
 - Device type: Raspberry Pi 4
 - OS: Picroft
 - Mycroft-core version: 21.2.0

**Additional context**
Seems like its getting an exception because its not understanding and then it breaks rather than the exception being handled, and saying that it doesnt understand it throws this error
@ChanceNCounter
Copy link
Contributor

Gonna need way more of the log file. Everything from the line that says "Traceback:" down.

@Arndorferd
Copy link
Author

Not sure what log file this is just on mycroft-cli-client output, voice.log has a similar amount of data with just

2021-08-21 00:00:08.145 | ERROR | 703 | mycroft.client.speech.listener:transcribe:239 | list index out of range
2021-08-21 00:00:08.148 | ERROR | 703 | mycroft.client.speech.listener:transcribe:240 | Speech Recognition could not understand audio

and no errors surrounding

@Arndorferd
Copy link
Author

there is no "Traceback:"

@ChanceNCounter
Copy link
Contributor

How about skills.log?

@Arndorferd
Copy link
Author

No errors in skills.log

@forslund
Copy link
Collaborator

That error occurs when the STT returns no response if I remember correctly. I.e Mycroft is invoked with "hey mycroft" but then nothing is said afterwards.

Not really an error but the case should be handled better

@ChanceNCounter
Copy link
Contributor

Yeah, but then 💀, which is different.

@forslund
Copy link
Collaborator

Ah yes definitely.

@Arndorferd
Copy link
Author

Well as someone who does a lot of bug reporting for open source, false lead warnings and errors need to be handled so that its easier for people to report real issues.

@JarbasAl
Copy link
Contributor

didnt @ken-mycroft mention fixing this issue some time back in chat?

@krisgesling
Copy link
Contributor

Yeah it's on the list to fix up - fully agree that false warnings make things harder so even if it's not "failing" as such, we still want to squish it.

@forslund
Copy link
Collaborator

forslund commented Sep 8, 2024

Closing Issue since we're archiving the repo

@forslund forslund closed this as completed Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants