Skip to content

[v640] Backport recent fixes to the release branch#22125

Merged
guitargeek merged 3 commits intoroot-project:v6-40-00-patchesfrom
guitargeek:bp_1
May 3, 2026
Merged

[v640] Backport recent fixes to the release branch#22125
guitargeek merged 3 commits intoroot-project:v6-40-00-patchesfrom
guitargeek:bp_1

Conversation

@guitargeek
Copy link
Copy Markdown
Contributor

No description provided.

bellenot and others added 3 commits May 3, 2026 08:56
Updated description of the Modified Anderson-Bjork root-finding algorithm to include proper citation.

(cherry picked from commit a1ae51b)
In `ROperator_Conv.hxx` line 202, the `+1` in the conv output formula `(input + pad - kernel) / stride + 1` was being string-concatenated onto the stride value instead of added separately, generating `((W+-3)/21)` for stride=2 instead of the correct `((W+-3)/2+1)`.

simple fix is to change the: `std::to_string(stride) + "1)";` to `+ std::to_string(stride) + "+1)";`

(cherry picked from commit 00cc11d)
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

Test Results

    22 files      22 suites   3d 9h 52m 33s ⏱️
 3 852 tests  3 851 ✅ 0 💤 1 ❌
76 064 runs  76 063 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 4dc53f2.

@guitargeek guitargeek merged commit d892c73 into root-project:v6-40-00-patches May 3, 2026
29 of 33 checks passed
@guitargeek guitargeek deleted the bp_1 branch May 3, 2026 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants