Skip to content

Conversation

@mzhou0
Copy link
Contributor

@mzhou0 mzhou0 commented Jun 25, 2021

This plugin categorizes a job based on its CPU performance. Adds a PCP archive for the integration test.

@mzhou0 mzhou0 marked this pull request as ready for review June 25, 2021 16:04
@mzhou0 mzhou0 marked this pull request as draft June 25, 2021 17:01
}
},
"cpucategories": {
"documentation": "The CpuCategories plugin uses CPU utilization metrics to categorize a job as good, pinned, unpinned, or low. A job is good if every core is heavily utilized, and a job is low if the cores are barely utilized or not at all. A pinned job consists of a scenario where a subset of the cores does most of the work, while an unpinned job is when the work is spread unevenly across many cores.",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"documentation": "The CpuCategories plugin uses CPU utilization metrics to categorize a job as good, pinned, unpinned, or low. A job is good if every core is heavily utilized, and a job is low if the cores are barely utilized or not at all. A pinned job consists of a scenario where a subset of the cores does most of the work, while an unpinned job is when the work is spread unevenly across many cores.",
"documentation": "Categorization of the CPU utilization of the job as good, pinned, unpinned, or low. A job is good if every core is heavily utilized, and a job is low if the cores are barely utilized or not at all. A pinned job consists of a scenario where a subset of the cores does most of the work, while an unpinned job is when the work is spread unevenly across many cores.",

self._deltas[node] = {}
self._maxcores[node] = 0

if proc is None:
Copy link
Member

Choose a reason for hiding this comment

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

pesudo suggestion

            if proc is None and 'cpusallowed' not in proc and node not in proc['cpusallowed'] . .....

@mzhou0 mzhou0 marked this pull request as ready for review July 8, 2021 14:32
@mzhou0 mzhou0 marked this pull request as draft July 8, 2021 16:59
@mzhou0 mzhou0 marked this pull request as ready for review July 8, 2021 17:17
@jpwhite4 jpwhite4 merged commit 5b4e382 into ubccr:master Jul 30, 2021
@mzhou0 mzhou0 deleted the cpucategories branch July 30, 2021 20:59
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