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

Create EnergyConsumption #694

Closed
wants to merge 1 commit into from

Conversation

rgopikrishnan91
Copy link
Contributor

No description provided.

@rgopikrishnan91
Copy link
Contributor Author

@kestewart @goneall Can you guys please review this its part 1 of the changes. Other PRs to follow


## Summary

Class that contains properties to describe EnergyConsumption incurred by a AI model in different stages of its lifecycle.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"to describe EnergyConsumption" -> "to describe energy consumption"


## Description

The class used for denoting the training energy consumption, inference energy consumption and fine tuning energy consumption of the AI model(s) used in a AI software.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Using "system" instead of "software"?

"used in a AI software" -> "used in an AI system"

Copy link
Contributor

@kestewart kestewart left a comment

Choose a reason for hiding this comment

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

Suggest applying the changes Art proposes, and merge into 1 PR with the other PRs.

Please also add your signed off on the revised PR.

@kestewart kestewart added this to the 3.0 milestone Apr 4, 2024
@kestewart kestewart added the Profile:AI Artificial intelligence profile label Apr 4, 2024
@rgopikrishnan91
Copy link
Contributor Author

Closing PR since this is now part of #697

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Profile:AI Artificial intelligence profile
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants