Skip to content
Snippets Groups Projects

CostMetric Code Implemented

Closed Syed Mafooq Ull Hassan requested to merge max-line-length into master
  • CostMetric implemented and tested (k-means algorithm implemented)
  • EnergyMetric implemented, but testing failed (using perf tool). Have to revisit the logic using static sleeper script to calculate CPU flops, which will further be used to calculate energy efficiency.

Merge request reports

Closed by Michael LoukerisMichael Loukeris 1 month ago (May 21, 2025 4:33pm UTC)

Merge details

  • The changes were not merged into master.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Hello @mafooq ,

    I would kindly ask you to recommit the changes you suggested so that we keep everything clear.

    As you’ll notice, I merged into the main branch only the commit that resolves issue #16. From there, I created two different branches—one for monetary cost and the other for energy efficiency. Since these are separate issues and attributes, let’s keep the development divided into these two individual branches.

    So, could you please commit the code for each attribute in the appropriate branch? It would also be very helpful if you could mention the issue number in your commit messages. For example: feat #4 (closed): Implemented the proposed solution for calculating the monetary cost. This will help keep the project history clear.

    Thank you!

    Edited by Michael Loukeris
Please register or sign in to reply
Loading