-
-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better exception handling and logging #17
Comments
Without thinking too much, I guess this issue may be related to the Exception thrown here: whisper-to-input/android/app/src/main/java/com/example/whispertoinput/WhisperTranscriber.kt Lines 76 to 78 in 6495487
The exception may cause the whisper-to-input/android/app/src/main/java/com/example/whispertoinput/WhisperInputService.kt Lines 114 to 123 in 6495487
A potential solution is to not throw an exception, but display a Toast message instead. This is just my random guess though... Another observation by @ijsun is that the amplitude feature still works when the issue occurs. |
Quick note: Displaying exceptions with toast messages are available after #23 |
As mentioned in the offline reported by @ijsun, the current keyboard will fail silently after using it for a while.
When encountered this issue, a simple
Clear All
to close all apps can reliably fix the issue by resetting the keyboard. Unfortunately, the steps to reproduce this issue is remained unknown. (not sure if it's due to unreliable internet connection)This kind of nondeterministic silent failure is difficult to debug. Therefore, we should somehow add a logging mechanism that can report all kinds of silent exceptions and maybe save it in a second page of the keyboard settings app, allowing the user to go through it for debugging purposes. (It's better to allow debugging directly on user's phone without the need of using
adb
or other external PC tools)Potentially related:
2023/12/02 Update: Clearing the cache in the app settings page can also fix the issue by resetting the keyboard.
The text was updated successfully, but these errors were encountered: