⚡ Bolt: Fast Dataframe Assignment & Increased Test Coverage

.Rbuildignore

@@ -22,3 +22,6 @@
 ^\.jules(/.*)?$
 ^\.trivyignore\.yaml$
 ^trivy\.yaml$
+^\.semgrepignore$
+^packrat$
+packrat/

.jules/bolt.md

@@ -16,3 +16,6 @@
 ## 2025-02-12 - R 언어에서 반복적인 mirt 모델 생성 시 불필요한 데이터프레임 부분집합 추출 최적화
 **Learning:** R에서 데이터프레임의 특정 열을 추출하는 작업(`df[cols]`)은 O(N)의 메모리 복사를 수반합니다. `autoFIPC`에서 `mirt` 모델의 파라미터를 설정하거나 호출하는 과정 중에 `newformXDataK[colnames(newFormModel@Data$data)]` 코드가 반복해서 사용되었고, 심지어 `ncol()`을 위해 단순히 개수를 구할 때도 사용되어 불필요한 메모리 할당과 오버헤드를 초래했습니다.
 **Action:** 조건문이나 반복문 내부에서 불필요하게 데이터프레임 부분집합 연산이 반복되지 않도록 외부에서 한 번만 `linkedFormData <- newformXDataK[colnames(newFormModel@Data$data)]`로 캐싱(caching)한 뒤, `ncol(linkedFormData)`와 `data = linkedFormData` 형태로 재사용하여 메모리 복사와 O(N) 오버헤드를 방지해야 합니다.
+## 2024-05-18 - Fast Dataframe Assignment in R
+**Learning:** In base R, 2D dataframe subsetting assignments (e.g. `df[idx, "col"] <- val`) suffer from significant `[<-.data.frame` method dispatch and dimensionality checking overhead.
+**Action:** Replace them with direct vector assignments (e.g. `df$col[idx] <- val`) to bypass the overhead for measurable performance gains in tight loops or large datasets.

R/aFIPC.R

@@ -598,15 +598,15 @@ autoFIPC <-
     # Preserve mirt's structural estimability flags. Forcing every row TRUE
     # frees boundary parameters such as 2PL g/u and makes the Hessian unstable.
 
-    NewScaleParms[NewScaleParms$item == 'GROUP', "est"] <- FALSE
-    OldScaleParms[OldScaleParms$item == 'GROUP', "est"] <- FALSE
+    NewScaleParms$est[NewScaleParms$item == 'GROUP'] <- FALSE
+    OldScaleParms$est[OldScaleParms$item == 'GROUP'] <- FALSE
 
-    NewScaleParms[NewScaleParms$name == "COV_11", "est"] <- TRUE
-    OldScaleParms[OldScaleParms$name == "COV_11", "est"] <- TRUE
+    NewScaleParms$est[NewScaleParms$name == "COV_11"] <- TRUE
+    OldScaleParms$est[OldScaleParms$name == "COV_11"] <- TRUE
 
     if (itemtype == 'Rasch') {
-      NewScaleParms[NewScaleParms$name == "a1", "est"] <- FALSE
-      OldScaleParms[OldScaleParms$name == "a1", "est"] <- FALSE
+      NewScaleParms$est[NewScaleParms$name == "a1"] <- FALSE
+      OldScaleParms$est[OldScaleParms$name == "a1"] <- FALSE
     }
 
     #IPD
@@ -786,14 +786,14 @@ autoFIPC <-
         oldIdx <- oldScaleParmsItemIdxCache[[oldFormItemStr]]
 
         # ⚡ Bolt: Remove unnecessary paste0() array string generation overhead
-        message('   Newform Parms: ', paste(NewScaleParms[newIdx, "value"], collapse = ' '))
-        message('   Oldform Parms: ', paste(OldScaleParms[oldIdx, "value"], collapse = ' '))
+        message('   Newform Parms: ', paste(NewScaleParms$value[newIdx], collapse = ' '))
+        message('   Oldform Parms: ', paste(OldScaleParms$value[oldIdx], collapse = ' '))
 
-        NewScaleParms[newIdx, "value"] <-
-          OldScaleParms[oldIdx, "value"]
-        message('   Linkedform Parms: ', paste(NewScaleParms[newIdx, "value"], collapse = ' '), '\n')
+        NewScaleParms$value[newIdx] <-
+          OldScaleParms$value[oldIdx]
+        message('   Linkedform Parms: ', paste(NewScaleParms$value[newIdx], collapse = ' '), '\n')
 
-        NewScaleParms[newIdx, "est"] <-
+        NewScaleParms$est[newIdx] <-
           FALSE
       } else {
         message(
@@ -813,17 +813,17 @@ autoFIPC <-
       newBetaIdx <- NewScaleParms$item == 'BETA'
       oldBetaIdx <- OldScaleParms$item == 'BETA'
 
-      NewScaleParms[newBetaIdx, "value"] <-
-        OldScaleParms[oldBetaIdx, "value"]
-      NewScaleParms[newBetaIdx, "est"] <-
+      NewScaleParms$value[newBetaIdx] <-
+        OldScaleParms$value[oldBetaIdx]
+      NewScaleParms$est[newBetaIdx] <-
         FALSE
 
       message('applying BETA parameter as linking')
 
       message(
         '   Linkedform Parms: ',
         paste0(
-          NewScaleParms[newBetaIdx, "value"],
+          NewScaleParms$value[newBetaIdx],
           ' '
         ),
         '\n'
@@ -858,13 +858,13 @@ autoFIPC <-
       new_mean11_idx <- NewScaleParms$name == "MEAN_11"
       old_mean11_idx <- OldScaleParms$name == "MEAN_11"
 
-      NewScaleParms[new_cov11_idx, "est"] <- FALSE
-      OldScaleParms[old_cov11_idx, "est"] <- FALSE
-      NewScaleParms[new_mean11_idx, "est"] <- FALSE
-      OldScaleParms[old_mean11_idx, "est"] <- FALSE
+      NewScaleParms$est[new_cov11_idx] <- FALSE
+      OldScaleParms$est[old_cov11_idx] <- FALSE
+      NewScaleParms$est[new_mean11_idx] <- FALSE
+      OldScaleParms$est[old_mean11_idx] <- FALSE
 
-      NewScaleParms[new_cov11_idx, "value"] <- 1
-      OldScaleParms[old_mean11_idx, "value"] <- 0
+      NewScaleParms$value[new_cov11_idx] <- 1
+      OldScaleParms$value[old_mean11_idx] <- 0
     }
     if (freeMEAN == T) {
       LinkedModelSyntax <-
@@ -875,8 +875,8 @@ autoFIPC <-
           'MEAN = F1'
         ))
 
-      NewScaleParms[NewScaleParms$name == "MEAN_1", "est"] <- TRUE
-      OldScaleParms[OldScaleParms$name == "MEAN_1", "est"] <- TRUE
+      NewScaleParms$est[NewScaleParms$name == "MEAN_1"] <- TRUE
+      OldScaleParms$est[OldScaleParms$name == "MEAN_1"] <- TRUE
     } else {
       LinkedModelSyntax <-
         mirt::mirt.model(paste0(

aFIPC.Rcheck/00_pkg_src/aFIPC/DESCRIPTION

@@ -0,0 +1,18 @@
+Package: aFIPC
+Type: Package
+Title: Automated Fixed Item Parameter Linking
+Version: 0.1.0
+Author: Seongho Bae [aut, cre]
+Maintainer: Seongho Bae <seongho@kw.ac.kr>
+Authors@R: person(given = "Seongho", family = "Bae", role = c("aut", "cre"),
+    email = "seongho@kw.ac.kr")
+Description: Automates fixed item parameter linking for test linking under
+    the item response theory paradigm using mirt package estimates.
+License: GPL-3 | file LICENSE
+Imports: mirt, methods
+Suggests: testthat (>= 3.0.0)
+Encoding: UTF-8
+Config/testthat/edition: 3
+Config/roxygen2/version: 8.0.0
+NeedsCompilation: no
+Packaged: 2026-07-31 16:50:43 UTC; jules

aFIPC.Rcheck/00_pkg_src/aFIPC/LICENSE

@@ -0,0 +1,2 @@
+YEAR: 2026
+COPYRIGHT HOLDER: Seongho Bae

aFIPC.Rcheck/00_pkg_src/aFIPC/NAMESPACE

@@ -0,0 +1,5 @@
+# Generated by roxygen2: do not edit by hand
+
+export(autoFIPC)
+export(surveyFA)
+import(mirt)