Think about how to have this class inherit from the other BLI2D classes

Member genie::BLI2DNonUnifObjectGrid< ZObject, IndexType, XType, YType >::get_bound_indices (const std::vector< Type > *vec, Type val, int &lower_index, int &upper_index) const

Check that the vector contains at least two entries

Document enum values

Document enum values

Member genie::NievesSimoVacasMECPXSec2016::XSec (const Interaction *i, KinePhaseSpace_t k) const

Replace these hard-coded replacements with an equivalent XML configuration

Revisit this assumption, and perhaps implement something more robust

Shouldn't we get this from the nuclear model?

Member genie::SuSAv2MECPXSec::Qvalue (const Interaction &interaction) const

Add more hadron tensors so this scaling is not so terrible

Switch to using std::unique_ptr here for easy cleanup once C++11 features are allowed in GENIE

Member genie::TabulatedLabFrameHadronTensor::dSigma_dT_dCosTheta (int probe_pdg, double E_probe, double m_probe, double Tl, double cos_l, double ml, double Q_value) const

Add check if in grid. If not, return 0

Add any needed changes for positron projectiles

Member genie::TabulatedLabFrameHadronTensor::dSigma_dT_dCosTheta_rosenbluth (int probe_pdg, double E_probe, double m_probe, double Tl, double cos_l, double ml, double Q_value) const

Add check if in grid. If not, return 0

Add any needed changes for positron projectiles

Replace these with lambdas (or something similar) when GENIE updates to C++11. This technique is a bit of a hack.

Add error checks

Use type name

Think about adding the imaginary part even though it's not needed for the cross section calculation