Class corsika::BaseProcess< TDerived >

rename BaseProcess into just Process

Member corsika::ConstStackIteratorInterface< TStackData, TParticleInterface, MSecondaryProducer, TStackType >::ConstStackIteratorInterface (ConstStackIteratorInterface &&rhs)

check rule of five

Member corsika::FourVector< TTimeType, TSpaceVecType >::isTimelike () const

FIXME: a better alternative would be to define an enumeration enum { SpaceLike =-1, TimeLike, LightLike } V4R_Category; and a method called V4R_Category GetCategory() const; RU: then you have to decide in the constructor which avoids "lazyness"

Member corsika::get_transformation (CoordinateSystem const &c1, CoordinateSystem const &c2)

make this a protected member of CoordinateSystem

Class corsika::HadronicElasticInteraction

add unit test

Namespace corsika::history

add options/parameters to 'detail::hist_factory()'

Member corsika::IMediumModel::getIntegratedGrammage (BaseTrajectory const &) const =0

think about the mixin inheritance of the trajectory vs the BaseTrajectory approach; for now, only lines are supported (?).

Class corsika::LongitudinalProfile

test missing

Class corsika::node::GeometryDataInterface< T, TEnvType >

fixme: Document type T

Member corsika::Point::getCoordinates () const

TODO: this should be private or protected, we don NOT want to expose numbers without reference to outside:

Class corsika::QuantityVector< TDimension >

review QuantityVector: can this be a protected-only inheritance to other objects? We don't want to expose access to low-level Eigen objects anywhere.

Class corsika::ShowerAxis

documentation needs update ...

Member corsika::StackIteratorInterface< TStackData, TParticleInterface, MSecondaryProducer, TStackType >::StackIteratorInterface (StackIteratorInterface &&rhs)

check rule of five

Member corsika::Vector< TDimension >::getComponents () const

this should best be protected, we don't want users to use bare coordinates without reference frame

Class corsika::WeightProviderIterator< AConstIterator, BConstIterator >

TODO: replace with https://www.boost.org/doc/libs/1_74_0/libs/iterator/doc/zip_iterator.html or ranges zip

check resource allocation