Skip to content
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

Doesn't display all ANSI correctly #1

Open
sairuk opened this issue Feb 10, 2016 · 5 comments
Open

Doesn't display all ANSI correctly #1

sairuk opened this issue Feb 10, 2016 · 5 comments
Assignees

Comments

@sairuk
Copy link
Owner

@sairuk sairuk commented Feb 10, 2016

Script seems trip up on some ANSI's then not reset itself properly for the next. While it will play others perfectly fine.

Need some more information on ANSI codes.

@sairuk
Copy link
Owner Author

@sairuk sairuk commented Feb 10, 2016

Examples of the problem

20160211_072535
20160211_072509
20160211_072446
20160211_072433

@sairuk sairuk changed the title Doesn't display all ANSI's correctly Doesn't display all ANSI correctly Feb 10, 2016
@sairuk
Copy link
Owner Author

@sairuk sairuk commented Feb 12, 2016

There is also a case where the newline will not write correctly, may be related to the above.

@sairuk
Copy link
Owner Author

@sairuk sairuk commented Feb 14, 2016

I have changed the escape code used from clear [2J the screen to reset [033c which appears in testing a much better solution, the terminal was switching modes and never resetting properly between ANSI

@sairuk sairuk self-assigned this Feb 14, 2016
@sairuk
Copy link
Owner Author

@sairuk sairuk commented Feb 15, 2016

Escape code change fixed issue as shown in images

@sairuk sairuk closed this Feb 15, 2016
@sairuk
Copy link
Owner Author

@sairuk sairuk commented Mar 4, 2016

@Commodork re-experienced this issue with v02 after an extended run. We don't know the triggering file yet though

pyans_corrupt_display_commodork

@sairuk sairuk reopened this Mar 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@sairuk and others