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.

Sign up for GitHub

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

Jump to bottom

SolverConfiguration.ClearConsole doesn't work properly #43

Closed
eduherminio opened this issue Dec 16, 2020 · 1 comment
Closed

SolverConfiguration.ClearConsole doesn't work properly #43

eduherminio opened this issue Dec 16, 2020 · 1 comment
Labels
bug Something isn't working

Comments

@eduherminio
Copy link
Owner

eduherminio commented Dec 16, 2020 •
edited
Loading

SolverConfiguration.ClearConsole doesn't work properly.

More specifically, it doesn't work at least when the output to clear is big (i.e. you've solved a bunch of days or are using SolverConfiguration.ShowConstructorElapsedTime or/and SolverConfiguration.ShowTotalElapsedTimePerDay), but sometimes in other scenarios as well.

It doesn't matter what combination of Console.Clear() and/or AnsiConsole.Console.Clear(true) is used, but I can't find a way to fully clear the output of previous runs.

Back buffer rendering technique mentioned in spectreconsole/spectre.console#156 to re-render the table doesn't fix this issue either.
@eduherminio eduherminio added the bug Something isn't working label Dec 16, 2020
@eduherminio eduherminio mentioned this issue Dec 16, 2020
Merged
eduherminio added a commit that referenced this issue Dec 16, 2020
@eduherminio
Measure constructor time (#44)
e3e02c9 
* Add `SolverConfiguration.ShowConstructorElapsedTime` to allow measuring constructor time (closes #39).
* Add `SolverConfiguration.ShowTotalElapsedTimePerDay` to aggregate constructor, part1 & part2 times (as it's done to calculate the mean time per day in the overall results panel).
#43 is a known bug when using both options, but isn't caused by them.
@eduherminio eduherminio mentioned this issue Jul 8, 2021
Closed
@eduherminio
Copy link
Owner Author

Fixed in #86, since Live Display Spectre Console feature implies no more console clearing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eduherminio