composable_kernel icon indicating copy to clipboard operation
composable_kernel copied to clipboard

Add IO module for profiler

Open smedegaard opened this issue 6 months ago • 2 comments

Problem Description

ck_profiler outputs its results to std::cout. Handling the output down stream therefore relies on string parsing which can be error prone and difficult to evolve.

Suggested Solution

Adding an IO module could enable reusable and extendable output functionality. This enhancement would improve the profiler's flexibility and maintainability by providing a dedicated module for handling various output formats and destinations.

Key objectives:

Create a modular IO system for the profiler Enable different output formats (e.g., plaintext, CSV, ...) Support multiple output destinations (file, stdout, custom handlers) Provide an extensible interface for adding new output formats/handlers System Information Operating System: Not hardware/system specific (Development feature) CPU: Not hardware specific (Development feature) GPU: Not hardware specific (Development feature) ROCm Version: Latest development version

Additional Information

This enhancement will benefit the profiler by:

Separating concerns between data collection and output handling Making it easier to add new output formats without modifying core profiler code Allowing users to implement custom output handlers for their specific needs Improving the maintainability of output-related code Enabling better integration with external tools and analysis pipelines

smedegaard avatar May 21 '25 06:05 smedegaard

@smedegaard, does self-assigned indicate you're working on this already?

AviralGoelAMD avatar May 23 '25 14:05 AviralGoelAMD

sorry for the late reply @AviralGoelAMD

I haven't started this officially.

smedegaard avatar Jun 03 '25 08:06 smedegaard