-
Notifications
You must be signed in to change notification settings - Fork 420
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
Scrambled spaces in logging #72
Comments
@michaelzhiluo Hi Michael, could you please describe the issue you observed? Is it during provision or executing the tasks? Is it happening for the multiple tasks scenario (for this one, we may need to set stream_logs to false to avoid multiple tasks shooting the outputs to the console simultaneously)? Thanks! |
I found a problem with the Ctrl-C. When the provision/task is running and printing out the results, sometimes, Ctrl-C does not work. We may need to capture the KeyboardInterrupt exception while redirecting the output. |
@Michaelvll Sorry just saw this! I am not observing this issue anymore (somehow it fixed itself) |
I reproduced the error. After Current TODOs:
|
Wow this is a really good find!
…On Fri, Dec 3, 2021 at 4:56 AM Zhanghao Wu ***@***.***> wrote:
There's also a situation where @infwinston <https://github.com/infwinston>
and I saw - sometimes after Ctrl-C, or after tailing the log, the lines
contain scrambled spaces (like we saw before) or the shell's cursor
disappears. Did you see that issue?
*Originally posted by @concretevitamin
<https://github.com/concretevitamin> in #69 (comment)
<#69 (comment)
I reproduced the error. After tail -f <sky_logs> when there is a progress
bar 'progressing' and Ctrl-C, the cursor will disappear. It may be due to
the \r produced by the progress bar.
Current TODOs:
- Fix cursor missing problem
- Fix Ctrl-C not responding problem
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#72 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEQWHRMJSDKB3YYFDEVODLUO7MQZANCNFSM5JANYQ4Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Fixed in #82. Please feel free to open a new issue if you find any problem. |
There's also a situation where @infwinston and I saw - sometimes after Ctrl-C, or after tailing the log, the lines contain scrambled spaces (like we saw before) or the shell's cursor disappears. Did you see that issue?
Originally posted by @concretevitamin in #69 (comment)
The text was updated successfully, but these errors were encountered: