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

benchmarks #3463

Merged
merged 3 commits into from
Nov 5, 2024
Merged

benchmarks #3463

merged 3 commits into from
Nov 5, 2024

Conversation

yegor256
Copy link
Member

@yegor256 yegor256 commented Nov 5, 2024

PR-Codex overview

This PR introduces benchmarking functionality to the eo-maven-plugin, adds a new StMeasured class for performance measurement, and updates the GitHub Actions workflow to automate benchmarking results collection and reporting.

Detailed summary

  • Added ## Benchmark section in README.md.
  • Introduced StMeasured class in eo-parser/src/main/java/org/eolang/parser/ for measuring performance.
  • Updated OptimizeMojo.java and TranspileMojo.java to use TrLambda with StMeasured.
  • Created a new GitHub Actions workflow in .github/workflows/benchmark.yml for automated benchmarking.
  • Adjusted README.md to include links to benchmark results.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@yegor256 yegor256 merged commit 39b718d into master Nov 5, 2024
15 of 19 checks passed
@yegor256 yegor256 deleted the 3257 branch November 5, 2024 17:50
@yegor256 yegor256 restored the 3257 branch November 5, 2024 17:50
@0crat
Copy link

0crat commented Nov 6, 2024

@yegor256 Hey there! 👋 Thanks for your contribution of 184 hits-of-code! You've earned +4 points, which breaks down like this: +4 base, +9 for your code, but -16 for no review and -16 for no comments. We bumped it up by +23 to give you something. Remember, code reviews and comments are crucial – they could've boosted your points significantly! Keep the contributions coming, and don't forget to balance speed and quality. Your current balance is -48. Looking forward to your next contribution! 💪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants