Skip to content

Add post-processing and integrate fb_power into perf hooks#500

Open
lrh12580 wants to merge 1 commit intofacebookresearch:v2-betafrom
lrh12580:export-D94542927-to-v2-beta
Open

Add post-processing and integrate fb_power into perf hooks#500
lrh12580 wants to merge 1 commit intofacebookresearch:v2-betafrom
lrh12580:export-D94542927-to-v2-beta

Conversation

@lrh12580
Copy link

@lrh12580 lrh12580 commented Mar 4, 2026

Summary:
The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM.

  • Task: T248954685
  • Claude CLI prompts:
    • design.md (P2203208599)
    • implementation.md (P2203210057)
  • Related older diffs:
    • D73953894
    • D92542871

Summary
Adds post-processing to the fb_power monitor and integrates it with perf hooks. The post-processing engine reads power.csv and computes results using platform-specific sensor mappings.

Changes

  • Add fb_power/post_process.py with PowerPostProcessor
  • Extend constants.py with per-platform sensor mapping config
  • Update perf.py to pass post_process to fb_power
  • Update BUCK to include post_process.py
  • Export PowerPostProcessor from __init__.py

Differential Revision: D94542927

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 4, 2026
@meta-codesync
Copy link

meta-codesync bot commented Mar 4, 2026

@lrh12580 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D94542927.

@lrh12580 lrh12580 force-pushed the export-D94542927-to-v2-beta branch from a074573 to 47c8013 Compare March 6, 2026 22:35
lrh12580 added a commit to lrh12580/DCPerf that referenced this pull request Mar 6, 2026
…esearch#500)

Summary:

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685  
- **Claude CLI prompts:**  
  - `design.md` (P2203208599)  
  - `implementation.md` (P2203210057)  
- **Related older diffs:**  
  - D73953894  
  - D92542871  

**Summary** Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and produces normalized outputs using **platform-specific sensor mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled/passed through
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
lrh12580 added a commit to lrh12580/DCPerf that referenced this pull request Mar 10, 2026
…esearch#500)

Summary:

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685  
- **Claude CLI prompts:**  
  - `design.md` (P2203208599)  
  - `implementation.md` (P2203210057)  
- **Related older diffs:**  
  - D73953894  
  - D92542871  

**Summary**  
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
@lrh12580 lrh12580 force-pushed the export-D94542927-to-v2-beta branch from 47c8013 to 3a8d324 Compare March 10, 2026 02:15
lrh12580 added a commit to lrh12580/DCPerf that referenced this pull request Mar 10, 2026
…esearch#500)

Summary:

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685  
- **Claude CLI prompts:**  
  - `design.md` (P2203208599)  
  - `implementation.md` (P2203210057)  
- **Related older diffs:**  
  - D73953894  
  - D92542871  

**Summary**  
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
@lrh12580 lrh12580 force-pushed the export-D94542927-to-v2-beta branch from 3a8d324 to 041f4d8 Compare March 10, 2026 15:42
lrh12580 added a commit to lrh12580/DCPerf that referenced this pull request Mar 10, 2026
…esearch#500)

Summary:
Pull Request resolved: facebookresearch#500

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685
- **Claude CLI prompts:**
  - `design.md` (P2203208599)
  - `implementation.md` (P2203210057)
- **Related older diffs:**
  - D73953894
  - D92542871

**Summary**
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
@lrh12580 lrh12580 force-pushed the export-D94542927-to-v2-beta branch from 041f4d8 to 7337c28 Compare March 10, 2026 15:45
lrh12580 added a commit to lrh12580/DCPerf that referenced this pull request Mar 10, 2026
…esearch#500)

Summary:

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685  
- **Claude CLI prompts:**  
  - `design.md` (P2203208599)  
  - `implementation.md` (P2203210057)  
- **Related older diffs:**  
  - D73953894  
  - D92542871  

**Summary**  
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
@lrh12580 lrh12580 force-pushed the export-D94542927-to-v2-beta branch from 7337c28 to 3b10a65 Compare March 10, 2026 15:46
lrh12580 added a commit to lrh12580/DCPerf that referenced this pull request Mar 10, 2026
…esearch#500)

Summary:
Pull Request resolved: facebookresearch#500

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685
- **Claude CLI prompts:**
  - `design.md` (P2203208599)
  - `implementation.md` (P2203210057)
- **Related older diffs:**
  - D73953894
  - D92542871

**Summary**
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
@lrh12580 lrh12580 force-pushed the export-D94542927-to-v2-beta branch from 3b10a65 to c73adfe Compare March 10, 2026 15:49
lrh12580 added a commit to lrh12580/DCPerf that referenced this pull request Mar 10, 2026
…esearch#500)

Summary:
Pull Request resolved: facebookresearch#500

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685
- **Claude CLI prompts:**
  - `design.md` (P2203208599)
  - `implementation.md` (P2203210057)
- **Related older diffs:**
  - D73953894
  - D92542871

**Summary**
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
@lrh12580 lrh12580 force-pushed the export-D94542927-to-v2-beta branch 2 times, most recently from d19ba6b to dfb24e5 Compare March 11, 2026 00:07
lrh12580 added a commit to lrh12580/DCPerf that referenced this pull request Mar 11, 2026
…esearch#500)

Summary:

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685  
- **Claude CLI prompts:**  
  - `design.md` (P2203208599)  
  - `implementation.md` (P2203210057)  
- **Related older diffs:**  
  - D73953894  
  - D92542871  

**Summary**  
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
…esearch#500)

Summary:
Pull Request resolved: facebookresearch#500

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685
- **Claude CLI prompts:**
  - `design.md` (P2203208599)
  - `implementation.md` (P2203210057)
- **Related older diffs:**
  - D73953894
  - D92542871

**Summary**
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927
@lrh12580 lrh12580 force-pushed the export-D94542927-to-v2-beta branch from dfb24e5 to aa63adb Compare March 11, 2026 00:11
meta-codesync bot pushed a commit that referenced this pull request Mar 11, 2026
Summary:
Pull Request resolved: #500

The diff stack enables power-telemetry tracing for Benchpress plugins and supports the following platforms: **T1_CPL, T1_MLN, T1_BGM, T1_TRN, and T11_GRC_ARM**.

- **Task:** T248954685
- **Claude CLI prompts:**
  - `design.md` (P2203208599)
  - `implementation.md` (P2203210057)
- **Related older diffs:**
  - D73953894
  - D92542871

**Summary**
Adds a **post-processing pipeline** to the `fb_power` monitor and wires it into **perf hooks** so power results can be computed automatically after collection. The post-processing engine reads `power.csv` and emits normalized outputs using **platform-specific sensor-mapping configuration**.

**Changes**
- Add `fb_power/post_process.py` implementing `PowerPostProcessor`
- Add/extend `constants.py` with per-platform sensor-mapping config used by post-processing
- Update `perf.py` to invoke `fb_power` with `post_process` enabled (plumb/forward the flag as needed)
- Update `BUCK` targets to include `post_process.py`
- Export `PowerPostProcessor` from `__init__.py` for external use

Differential Revision: D94542927

fbshipit-source-id: 2f673ec0ae11cc862b7363963cb97b3adcb5d2c6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant