beyond-all-reason / beyond-all-reason/RecoilEngine

Check math.round perf

Open
#2,373 3 comments 0 reactions 0 assignees View on GitHub
refactor
Dominant language
C++
Stars
679
Forks
290
Avg merge
3d 2h
Merged PRs (30d)
40

Description

Here's a surprising claim that math.round(x) is measurably slower than math.floor(x + 0.5)
https://github.com/beyond-all-reason/Beyond-All-Reason/pull/5246#discussion_r2135422692

Figure out what's going on, how bad it is and if we can do anything about it.

Contributor guide

Open the contributing guide

Research direction

Start with the linked pull-request discussion and reproduce the reported comparison between math.round(x) and math.floor(x + 0.5) using a representative benchmark. Quantify whether the difference is reproducible and significant, then document the cause and whether a change is justified.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
performance
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.