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

Show dynamically updating OCGV while pipeline input is being received #209

Open
maskati opened this issue Aug 7, 2023 · 4 comments
Open
Labels
bug Something isn't working OCGV

Comments

@maskati
Copy link

maskati commented Aug 7, 2023

Summary of the new feature / enhancement

Compare 1..5|%{Start-Sleep -Seconds 1;$_}|Out-GridView with 1..5|%{Start-Sleep -Seconds 1;$_}|Out-ConsoleGridView. OGV displays records as they are received (ProcessRecord) while OCGV shows records only at the end (EndProcessing).

Proposed technical implementation details (optional)

No response

@tig tig added bug Something isn't working OCGV labels Aug 7, 2023
@tig
Copy link
Collaborator

tig commented Aug 7, 2023

Thanks @maskati. I didn't realize OGV did that!

Can you explain the use case for having the results show up dynamically?

@BDisp
Copy link

BDisp commented Aug 7, 2023

@tig I think this may be a choice to change how OCGV display the results. Instead displaying on the same terminal, it can be displayed on a separate terminal through using the ProcessStartInfo. If you using various OGV commands it'll open on separated terminals.

@maskati
Copy link
Author

maskati commented Aug 7, 2023

@tig anything where you are handling larger or slower to enumerate datasets, and when using the grid view as a filterable view of such records. See for example Stream Application Insights live metrics to a local PowerShell grid view.

@tig
Copy link
Collaborator

tig commented Aug 7, 2023

@tig anything where you are handling larger or slower to enumerate datasets, and when using the grid view as a filterable view of such records. See for example Stream Application Insights live metrics to a local PowerShell grid view.

Thanks for this!

I'll take a closer look at how hard it would be to change OCGV to support this use case. I can't imagine any blockers.

Terminal.Gui now supports a TableView that was designed to support very large data sets, loaded dynamically. When OCGV was first developed this didn't exist and thus it uses ListView which is not great with huge amounts of data, and not ideally suited for dynamic updating. I have an ambition to re-factor OCGV to use TableView. This provides more motivation!

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

No branches or pull requests

3 participants