-
Notifications
You must be signed in to change notification settings - Fork 1
Refresh Dec 04 benchmark snapshot #90
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
Merged
NewYaroslav
merged 17 commits into
main
from
codex/verify-readme-and-documentation-completeness
Dec 5, 2025
Merged
Refresh Dec 04 benchmark snapshot #90
NewYaroslav
merged 17 commits into
main
from
codex/verify-readme-and-documentation-completeness
Dec 5, 2025
Conversation
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
Rerun LOGIT_BENCH_TOTAL=10000 with the latest spdlog adapter and update the Dec 04 snapshot CSV plus benchmark tables in README (EN/RU) and Doxygen.
Optimize spdlog pointer encoding to avoid iostream overhead, reduce async null copies in the LogIt adapter, and refresh the documented benchmark snapshot to match the latest run.
Add an opt-in passthrough flag on ILogFormatter and route passthrough formatters directly through Logger without extra string copies. Update the benchmark passthrough formatter to advertise the fast path.
Update benchmark snapshot timestamps and takeaways across README files and Doxygen after rerunning LOGIT_BENCH_TOTAL=10000.
Update benchmark snapshot bullet points to the latest 10k run at 06:53 UTC and keep README/Doxygen tables pointing to the refreshed CSV.
Add LatencyRecorder and user_data usage notes to README, README-RU, and Doxygen main page. Expose an optional user_data payload in LogRecord for benchmark adapters and document how to pass tokens without args_array copies.
Removed user_data member from LogRecord structure.
Changed args_array to be mutable for caching purposes.
Update README, README-RU, and Doxygen benchmark tables to the latest Dec 05 03:18 UTC run.
Update logger snapshot handling and refresh benchmark docs
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
Testing
cmake -S . -B build -DCMAKE_BUILD_TYPE=Release -DLOGIT_BENCH_ENABLE=ON -DLOGIT_BENCH_WITH_SPDLOG=ONcmake --build build --target logit_bench -jLOGIT_BENCH_TOTAL=10000 ./build/logit_benchCodex Task