Small inconsistency in the `lmBenchmark.R` file?

Offen

#126 erstellt am 12.6.2023 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: cpp, r
Bereich: documentation

Rechercherichtung

Compare the method mapping in lmBenchmark.R with the documented RcppEigen::fastLm method values, starting from the shown exprs["LDLt"] entry and the fastLm documentation. Verify which value represents LDLT and update the inconsistent source or documentation; done when the benchmark and documentation agree.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

In the lmBenchmark.R file I read

    ## LDLt Cholesky decomposition with rank detection
    exprs["LDLt"] <- alist(RcppEigen::fastLmPure(mm, y, 2L))

but in the documentation for RcppEigen::fastLm the method is described as

an integer scalar with value [...] 3 for the LDLT Cholesky,

Is this correct?

> sessionInfo()
R version 4.2.0 (2022-04-22)
Platform: aarch64-apple-darwin20 (64-bit)
Running under: macOS 13.3.1

Matrix products: default
LAPACK: /Library/Frameworks/R.framework/Versions/4.2-arm64/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods  
[7] base     

other attached packages:
[1] bench_1.1.3         RcppEigen_0.3.3.9.3 speedglm_0.3-5     
[4] biglm_0.9-2.1       DBI_1.1.3           MASS_7.3-58.1      
[7] Matrix_1.5-3

Vorherrschende Sprache: C++
Sterne: 118
Forks: 43
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus RcppCore/RcppEigen

Warn or error if Rcpp.h (and R.h) was included Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

RcppCore/RcppEigen#142 · Erstellt am 05.11.2025
setFromTriplets function gets segfault error Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

RcppCore/RcppEigen#136 · 5 Kommentare · Erstellt am 15.3.2024
RcppEigen Submatrix Views Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 20/100

RcppCore/RcppEigen#134 · 2 Kommentare · Erstellt am 11.2.2024
Map dgCmatrix as integer sparse matrix Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

RcppCore/RcppEigen#132 · 6 Kommentare · Erstellt am 19.10.2023
EIGEN_DEFAULT_TO_ROW_MAJOR and other flags Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

RcppCore/RcppEigen#118 · 3 Kommentare · Erstellt am 30.11.2022

Alle Issues in RcppCore/RcppEigen