Use `Eigen::BlockSparseMatrix` in Embedded Laplace approximation internals

Ouverte
#3,422 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
55/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
Active
Stack technique
cpp
Domaine
backend

Piste de recherche

Start in stan/math/mix/functor/laplace_marginal_density_estimator.hpp at the W_r definition around line 604. Read the referenced Eigen merge request and inspect how the embedded Laplace marginal density estimator functions use W_r. Done means those internals use Eigen::BlockSparseMatrix while preserving the existing estimator behavior.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

code cleanup performance

Eigen 5+ has BlockSparseMatrix which we can use for the block diagonal matrix W_r in the internals for the Laplace marginal density estimator functions.

https://gitlab.com/libeigen/eigen/-/merge_requests/2644

Langage dominant
C++
Étoiles
845
Forks
225
Merge moyen
4 j 21 h
PR mergées (30 j)
13

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de stan-dev/math

Toutes les issues de stan-dev/math

Issues similaires

Plus d'issues C++

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.