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

changes about hardware memory discussed during the hackathon #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sarahoury
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@dfovet dfovet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ca me semble bien conforme a nos échanges et remarques lors du hackaton
Au delta des remarques mineures a prendre en compte.

},
"nbComponent": {
"type": "integer",
"description": "the number of items of this component in your infrastructure"
},
"memorySize": {
"type":"integer",
"description": "the size of the memory of the component in Gbytes"
"description": "the size of the memory of the component in Gbytes, useful to detail the memory associated to ONE of your gpus for example (if we want the total memory, we will multiply the memorySize by nbComponent)."
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ici j'aurais ajouté : if the component is CPU do not put here the RAM size, create another component for the RAM this field is for the embeded memory of a component.

@@ -27,7 +31,7 @@
},
"series": {
"type": "string",
"description": "the series of this component, example: rtx4080"
"description": "the series of this component, example: gtx1080"
},
"share": {
"type": "number",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ici j'ajouterai : This share properties should be by defaut to 100% and if you do share the component indicate the percentage of share.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants