Add post-processing and integrate fb_power into perf hooks#500
Open
lrh12580 wants to merge 1 commit intofacebookresearch:v2-betafrom
Open
Add post-processing and integrate fb_power into perf hooks#500lrh12580 wants to merge 1 commit intofacebookresearch:v2-betafrom
lrh12580 wants to merge 1 commit intofacebookresearch:v2-betafrom
Conversation
a074573 to
47c8013
Compare
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
47c8013 to
3a8d324
Compare
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
3a8d324 to
041f4d8
Compare
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
041f4d8 to
7337c28
Compare
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
7337c28 to
3b10a65
Compare
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
3b10a65 to
c73adfe
Compare
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
d19ba6b to
dfb24e5
Compare
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
dfb24e5 to
aa63adb
Compare
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.
design.md(P2203208599)implementation.md(P2203210057)Summary
Adds post-processing to the
fb_powermonitor and integrates it with perf hooks. The post-processing engine readspower.csvand computes results using platform-specific sensor mappings.Changes
fb_power/post_process.pywithPowerPostProcessorconstants.pywith per-platform sensor mapping configperf.pyto passpost_processtofb_powerBUCKto includepost_process.pyPowerPostProcessorfrom__init__.pyDifferential Revision: D94542927