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

Revert "Rewrite the exponential decay process in C++. (#285)" #1

Closed
wants to merge 1 commit into from

Conversation

plietar
Copy link
Owner

@plietar plietar commented May 16, 2024

This reverts commit b3376d3.

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

Copy link

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 9f80999 is merged into master:

  • ❗🐌large_population: 7.47s -> 7.7s [+0.71%, +5.39%]
  • ✔️midrange_population: 913ms -> 928ms [-1.8%, +4.98%]
  • ✔️small_population: 308ms -> 310ms [-3.8%, +4.9%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

Copy link

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 9f80999 is merged into master:

  • ❗🐌large_population: 7.47s -> 7.7s [+0.71%, +5.39%]
  • ✔️midrange_population: 913ms -> 928ms [-1.8%, +4.98%]
  • ✔️small_population: 308ms -> 310ms [-3.8%, +4.9%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

Copy link

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 28556bd is merged into master:

  • ✔️large_population: 7.31s -> 7.29s [-3.29%, +2.96%]
  • ✔️midrange_population: 880ms -> 891ms [-4.69%, +7.27%]
  • ✔️small_population: 300ms -> 295ms [-11.74%, +8.35%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

2 similar comments
@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

Copy link

Hello

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 16, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 17, 2024

/benchmark

Copy link

This is how benchmark results would change (along with a 95% confidence interval in relative change) if c2aa43a is merged into master:

  • ✔️large_population: 7.49s -> 7.47s [-2.05%, +1.38%]
  • ✔️midrange_population: 912ms -> 910ms [-5.12%, +4.67%]
  • ✔️small_population: 306ms -> 304ms [-4.82%, +3.51%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@plietar
Copy link
Owner Author

plietar commented May 17, 2024

/benchmark

Copy link

This is how benchmark results would change (along with a 95% confidence interval in relative change) if c2aa43a is merged into master:

  • ✔️large_population: 7.22s -> 7.24s [-2.49%, +2.81%]
  • ✔️midrange_population: 849ms -> 846ms [-6.38%, +5.89%]
  • ✔️small_population: 257ms -> 252ms [-6.52%, +2.47%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

@plietar plietar force-pushed the master branch 6 times, most recently from d80b5ae to 719493b Compare May 17, 2024 15:16
@plietar
Copy link
Owner Author

plietar commented May 17, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 17, 2024

/benchmark

@plietar
Copy link
Owner Author

plietar commented May 17, 2024

/benchmark

Copy link

This is how benchmark results would change (along with a 95% confidence interval in relative change) if ac4c6f1 is merged into master:

  • ✔️large_population: 7.05s -> 7.07s [-1.75%, +2.41%]
  • 🚀midrange_population: 832ms -> 825ms [-1.52%, -0.02%]
  • ✔️small_population: 237ms -> 237ms [-2.46%, +2.24%]
    Further explanation regarding interpretation and methodology can be found in the documentation.

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.

1 participant