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

Fix multi gpu not on separate lines #127

Merged
merged 8 commits into from
Feb 19, 2022
Merged

Conversation

HO-COOH
Copy link
Contributor

@HO-COOH HO-COOH commented Feb 4, 2022

This is how neofetch showing multiple gpus
image

This is how winfetch showing multiple gpus before this change
image

This is after this change
image

@rashil2000
Copy link
Member

Please do not make formatting changes in the PR. Only change the working code.

@HO-COOH
Copy link
Contributor Author

HO-COOH commented Feb 4, 2022

Okay. Didn't notice vscode reformat something.

@rashil2000
Copy link
Member

Winfetch already supports multiline output. You should modify the info_gpu function to handle multiple GPUs better. See the info_colorbar to see how it's done.

What is the output of (Get-CimInstance -ClassName Win32_VideoController -Property Name).Name on your system?

@rashil2000
Copy link
Member

There are still some whitespace changes in the bottom part, kindly revert them

winfetch.ps1 Outdated Show resolved Hide resolved
winfetch.ps1 Outdated Show resolved Hide resolved
winfetch.ps1 Outdated Show resolved Hide resolved
Co-authored-by: Rashil Gandhi <46838874+rashil2000@users.noreply.github.com>
@rashil2000 rashil2000 merged commit f759d8b into lptstr:master Feb 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants