Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Fix wrong nvidia gpu card number #5353

Merged
merged 3 commits into from
Mar 8, 2021
Merged

Fix wrong nvidia gpu card number #5353

merged 3 commits into from
Mar 8, 2021

Conversation

hzy46
Copy link
Contributor

@hzy46 hzy46 commented Mar 5, 2021

Sometime nvidia_gpu_count.stdout_lines is ["<num>"], but sometimes it becomes ["", "<num>"].

Use |join("")|trim to unify them.

@hzy46 hzy46 mentioned this pull request Mar 5, 2021
Closed
@hzy46 hzy46 requested a review from suiguoxin March 5, 2021 09:59
@coveralls
Copy link

coveralls commented Mar 5, 2021 •
edited
Loading

Coverage Status

Coverage remained the same at 34.07% when pulling 99be5b2 on zhiyuhe/fix_nvidia_gpu_count into f0dad8b on master.

@hzy46 hzy46 merged commit 648d1bc into master Mar 8, 2021
@hzy46 hzy46 deleted the zhiyuhe/fix_nvidia_gpu_count branch March 8, 2021 04:11
@hzy46 hzy46 mentioned this pull request Mar 8, 2021
Closed
29 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@suiguoxin suiguoxin suiguoxin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hzy46 @coveralls @suiguoxin