-
Notifications
You must be signed in to change notification settings - Fork 28
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
Duplicate configuration.gpu.memory in aws.csv #226
Comments
Thanks for this bug report. We fixed it in the dev version: https://github.com/Boavizta/boaviztapi/blob/dev/boaviztapi/data/archetypes/cloud/aws.csv The dev version should be released soon. I recommend you to use it as of today. GPU data is not used because we don't yet have a method to take their impacts into account. Dedicated issue : #65 |
@da-ekchajzer OMG I have been using main for so long and wondered why so much seemed outdated. I am happy that you shared with me that you have a dev branch. Will use that going forward 🚀 |
Dev doc : dev.doc.api.boavizta.org/ Since we made some breaking changes, here is a draft of the pre-release : v1.0.0New features
Internal changes
Breaking changesCloud routersBefore
Now
Duration & allocationBefore
Now
Impacts formatBefore"gwp": {
"manufacture": 1900,
"use": 260,
"unit": "kgCO2eq"
}, or "manufacture_impacts": {
"gwp": {
"value": 23.8,
"unit": "kgCO2eq"
},
"pe": {
"value": 353,
"unit": "MJ"
},
"adp": {
"value": 0.02,
"unit": "kgSbeq"
}
}
} Now
"impacts": {
"gwp": {
"embedded": {
"value": 6.68,
"min": 6.68,
"max": 6.68,
"warnings": [
"End of life is not included in the calculation"
]
},
"use": "not implemented",
"unit": "kgCO2eq",
"description": "Total climate change"
}, Verbose formatBefore"USAGE": {
...
"usage_impacts": {
...
}
"avg_power": {
"value": 94.62364134445255,
"unit": "W",
"status": "COMPLETED",
"source": null
},
...
} Now
"CPU-1": {
"impacts": {
"gwp": {
"embedded": {
"value": 64.7,
"significant_figures": 3,
"min": 24.6,
"max": 149,
"warnings": [
"End of life is not included in the calculation"
]
},
"use": {
"value": 160,
"significant_figures": 2,
"min": 160,
"max": 160
},
"unit": "kgCO2eq",
"description": "Total climate change"
}
},
"die_size_per_core": {
"value": 0.47078947368421054,
"status": "COMPLETED",
"unit": "mm2",
"source": "Average for Skylake",
"min": 0.07,
"max": 1.02
}
...
} ContributorsToDo Known future requirements
|
Bug description
In
aws.csv
the data contains two columns with the nameconfiguration.gpu.memory
. Looking at the data, maybe the first instance should have been named toconfiguration.gpu.name
?boaviztapi/boaviztapi/data/devices/cloud/aws.csv
Line 1 in 5b6ef42
To Reproduce
Expected behavior
JSON OUTPUT
Additional context
The text was updated successfully, but these errors were encountered: