Move Eigen.hpp out of prim

Ouverte
#3,128 2 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é
3/5
Temps estimé
1-2 jours
Accessibilité débutants
45/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
cpp
Domaine
tooling

Piste de recherche

Commencez par stan/math/prim/fun/Eigen.hpp et recherchez dans le dépôt les inclusions de ce chemin ainsi que les références à l’emplacement du header. Déplacez le header vers le dossier math, mettez à jour les inclusions concernées en respectant la règle d’ordre fwd/rev, et vérifiez que le projet se compile et que ses tests réussissent.

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

Description

Description

Right now most files include stan/math/prim/fun/Eigen.hpp before everything else.

#include <stan/math/prim/fun/Eigen.hpp>
#include <stan/math/rev/meta.hpp>
#include <stan/math/rev/core.hpp>
#include <stan/math/rev/fun/value_of.hpp>

But our rules for fwd and rev are the you should always include fwd/rev files before including anything from prim. So we should move Eigen.hpp up to the math folder so it is out of prim.

Current Version:

v4.9.0

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.