LSSTDESC / firecrown

DESC Cosmology Likelihood Framework
BSD 3-Clause "New" or "Revised" License
29 stars 7 forks source link

DerivedParameter base class is not useful #373

Closed marcpaterno closed 7 months ago

marcpaterno commented 7 months ago

The abstract class DerivedParameter has only a single derived class. It seems to be a needless complication; the functionality of the one derived class DerivedParameterScalar apparently sufficient. We should collapse the functionality of DerivedParameterScalar into DerivedParameter and be rid of the inheritance.

DerivedParameterScalar is used in none of the LSST DESC repositories except for firecrown.