Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AMReX Tiny Profiler Context #54

Open
ax3l opened this issue Aug 3, 2022 · 1 comment
Open

AMReX Tiny Profiler Context #54

ax3l opened this issue Aug 3, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@ax3l
Copy link
Member

ax3l commented Aug 3, 2022

As users start to extend routines on the Python side - and we add more wrappers on bindings - we should start profiling these sections to catch unexpected bottlenecks.
For that, we should wrap and expose the BL_PROFILE logic.

BL_PROFILE are lightweight macros that wrap C++ object that start/stop profiling. We might want to wrap the underling C++ objects directly first. Then, we might add some convenience helper functions/objects as well, if needed.

@ax3l ax3l added the enhancement New feature or request label Aug 3, 2022
@ax3l
Copy link
Member Author

ax3l commented Jun 6, 2023

This could be implemented similar to cupy's profiler with an __enter__/__exit__ that can be used within a with: context manager (and thus does not rely on destructor calls for the time measurement).

Example:

@ax3l ax3l changed the title Profiling TIny Profiler Context Jun 6, 2023
@ax3l ax3l changed the title TIny Profiler Context AMReX Tiny Profiler Context Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant