stan-dev/stan

Remove unused variational::base_family class

Open

#1695 aperta il 19 nov 2015

Vedi su GitHub
 (6 commenti) (0 reazioni) (1 assegnatario)R (270 fork)batch import
code cleanupgood first issue

Metriche repository

Star
 (1476 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The base class variational::base_family has no virtual methods and the one function it does define uses no member variables or functions, so can be made standalone outside of the class. Therefore, base_family should be removed.

  • remove src/stan/variational/base_family.hpp
  • move function write_error_msg_() into its own file and rename without suffix _
  • patch families/normal_meanfield.hpp and families/normal_fullrank.hpp:
    • remove references to base_family
    • change calls to write_error_msg_ to include and call new function

If the intent was to somehow pull down some of the functionality from the subclasses, then this issue should be repurposed to do that.

Guida contributor

Direzione di ricerca
Rimuovi la classe base family e il suo header, sposta write error msg () in un file dedicato e aggiorna gli include nelle classi derivate.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80