Skip to content
This repository was archived by the owner on Mar 27, 2024. It is now read-only.

Feat(nvidia-smi): rewrite nvidia-smi module with native C API#318

Closed
Ehekatl wants to merge 1 commit into
netdata:masterfrom
Ehekatl:develop-dylan
Closed

Feat(nvidia-smi): rewrite nvidia-smi module with native C API#318
Ehekatl wants to merge 1 commit into
netdata:masterfrom
Ehekatl:develop-dylan

Conversation

@Ehekatl
Copy link
Copy Markdown

@Ehekatl Ehekatl commented Jan 9, 2020

not sure if it's acceptable...
this PR enables CGO, and embedded 3rd party lib directly under pkg.
I couldn't use go mod from source because I made some changes there (to fix nil point issue, etc
Also the chart names are slightly different from python version.

Other than that everything works well, I've being use this for quite long.

@tycho
Copy link
Copy Markdown

tycho commented Jan 9, 2020

Nice! This looks way better than my slow/kludgy Python garbage.

@ilyam8
Copy link
Copy Markdown
Member

ilyam8 commented Jan 10, 2020

hi @Ehekatl

this PR enables CGO

that is what we are trying to avoid, i removed nvidia and oracledb modules because of that

#204

I made some changes there

Why do not contribute to the upstream?

@Ehekatl
Copy link
Copy Markdown
Author

Ehekatl commented Jan 16, 2020

Ya I know @ilyam8
That's why I didn't expect to merge this. Just add it here in case someone need it, feel free to close

@ilyam8 ilyam8 added the wontfix This will not be worked on label Jan 22, 2020
@ilyam8
Copy link
Copy Markdown
Member

ilyam8 commented Mar 9, 2020

@ilyam8 ilyam8 closed this Mar 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants