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

Create basic performance measurement interface and integrate Caliper #51

Closed

Conversation

ilumsden
Copy link
Collaborator

This PR does three things. First, it creates a super basic interface for working with annotation-based performance measurement tools. This interface simply consists of two macros DYAD_PERF_REGION_BEGIN and DYAD_PERF_REGION_END. Second, this PR adds instances of these macros throughout the DTL and DYAD's core library. Finally, this PR adds an implementation of this interface for Caliper. This Caliper integration has been tested with the workflow I/O benchmark we are developing.

@ilumsden ilumsden added the enhancement New feature or request label Oct 11, 2023
@ilumsden ilumsden self-assigned this Oct 11, 2023
@ilumsden ilumsden force-pushed the caliper_integration_benchmark branch from 43800bf to 4aaf093 Compare November 23, 2023 02:06
@ilumsden ilumsden force-pushed the caliper_integration_benchmark branch 3 times, most recently from 4fe3643 to 11d5eef Compare December 13, 2023 15:27
1. Removed dyad_code from public headers
2. used PROJECT_NAME variable for all code
3. prefixed PROJECT_NAME for util code to avoid conflicts
@ilumsden ilumsden force-pushed the caliper_integration_benchmark branch 3 times, most recently from 464af40 to 6f02b9e Compare December 18, 2023 21:38
@ilumsden ilumsden force-pushed the caliper_integration_benchmark branch from 6f02b9e to 08abc06 Compare December 18, 2023 21:41
@ilumsden ilumsden mentioned this pull request Jan 29, 2024
3 tasks
@ilumsden ilumsden closed this Feb 28, 2024
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

Successfully merging this pull request may close these issues.

2 participants