-
Notifications
You must be signed in to change notification settings - Fork 4
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
replace has gpu with num gpu #211
Conversation
@@ -70,7 +80,7 @@ func nodeFromDBNode(info db.Node) types.Node { | |||
RentedByTwinID: uint(info.RentedByTwinID), | |||
SerialNumber: info.SerialNumber, | |||
Power: types.NodePower(info.Power), | |||
HasGPU: info.HasGPU, | |||
NumGPUs: getNumGPUs(info.HasGPU), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be NumGPU not NumGPUs similar to runtime.NumCPU https://pkg.go.dev/runtime#NumCPU
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the field will be num_gpus
right? I'd rather keep them consistent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, let's stay indeed consistent with the field in tfchain, i updated the title to make it num_gpu
threefoldtech/tfchain#756
ab15ba9
to
5d0e091
Compare
5d0e091
to
3a7e555
Compare
Changes
Related Issues
#210
Checklist