Skip to content

Conversation

@baltzell
Copy link
Collaborator

good for #756

@baltzell
Copy link
Collaborator Author

baltzell commented Aug 1, 2025

Good enough for now, from the unit test logs:

********************************************************************************************************************
*     BENCHMARK  RESULTS 
********************************************************************************************************************
*   TIMER (IT:GIV      ) : N Calls         9999,  Total Time  =         7.09 sec,  Unit Time =        0.710 msec   *
********************************************************************************************************************

Copy link
Member

@c-dilks c-dilks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Someday we should generalize these benchmark names, e.g., with this.getClass(); "IT:GIV" is a bit cryptic for an outsider. If benchmark pretty-printing is a concern, just publish a Markdown table and use the full benchmark name.

@c-dilks c-dilks merged commit 396aea4 into development Aug 1, 2025
34 checks passed
@c-dilks c-dilks deleted the 755-baltzell branch August 1, 2025 17:34
c-dilks pushed a commit that referenced this pull request Sep 15, 2025
tongtongcao pushed a commit that referenced this pull request Sep 22, 2025
tongtongcao pushed a commit that referenced this pull request Nov 18, 2025
tongtongcao pushed a commit that referenced this pull request Nov 21, 2025
tongtongcao pushed a commit that referenced this pull request Dec 5, 2025
tongtongcao pushed a commit that referenced this pull request Dec 5, 2025
@baltzell baltzell added the speed label Dec 23, 2025
tongtongcao pushed a commit that referenced this pull request Dec 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants