⚡ Bolt: Optimize ifelse with vectorized subsetting in llcont.glm

[seonghobae]

Superseded by #88

The valid product intent here is the same grouped-binomial llcont.glm() replacement of eager ifelse() calls with conditional preallocation/subsetting. Canonical #88 now carries that behavior on current protected master and also repairs the scalar-weight row-shape defect, adds a production-path regression, and preserves the unrelated hurdle benchmark instead of deleting it.

#88 exact head: fd746998ace024b36e3418fb7a4c9998023eff40

This branch's benchmark_hurdle_ifelse.R deletion is not a valid part of the GLM change, and its res_wt <- wt * 0 implementation does not safely cover a multi-row grouped response when wt is scalar. No unique valid semantic/test/fixture/contract delta remains outside #88, so this close is verified-successor consolidation rather than abandonment.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f7618929-0504-4ff0-998a-6e48449cd961

📥 Commits

Reviewing files that changed from the base of the PR and between 807e940 and 3ddec2a.

📒 Files selected for processing (2)

• R/llcont.R
• benchmark_hurdle_ifelse.R

💤 Files with no reviewable changes (1)

• benchmark_hurdle_ifelse.R

Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

llcont.glm의 이항 분포 계산이 사전 할당과 조건부 벡터 대입을 사용하도록 변경되었습니다. 관련 성능 벤치마크 파일과 함수가 삭제되었습니다.

Changes

이항 계산 최적화

Layer / File(s)	Summary
조건부 비율 및 가중치 계산 R/llcont.R, benchmark_hurdle_ifelse.R	llcont.glm은 0이 아닌 시행 횟수에만 반응 비율을 계산합니다. 양의 m을 가진 관측치에만 wt / m을 대입합니다. 기존 성능 벤치마크 파일과 함수는 삭제되었습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 3ddec

This is a localized performance optimization in the binomial calculation path, and no actionable merge-blocking risk remains after normal checks and review.

Possibly related PRs

• ContextualWisdomLab/nonnest2#88: 동일한 llcont.glm 이항 ifelse 최적화와 벤치마크 파일 삭제를 다룹니다.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 llcont.glm의 ifelse 호출을 벡터화된 부분집합 방식으로 최적화하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bolt/optimize-ifelse-llcont-glm-18026114900001235759

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}