Wrong conversion to Coulombs
Description
I think that I found a bug in units conversion in src/core/Allpix.cpp to Coulombs. The problem is in conversion number to Coulombs which I suppose should not be 1.6021766208e-19, but 6.241509126e18 (which is 1/1.6021766208e-19).
I think that this conversion has never been used in entire framework so far, so nobody noticed. I realized this when writing a module which is able to perform threshold scan, which we use often in ITk strips testing, and we usually use fC when talking about a charge collected from detector.
Proposal
By this I would also like to propose adding unit fC to units in src/core/Allpix.cpp in "charge section". I think it could look something like this:
// CHARGE
Units::add("e", 1);
Units::add("ke", 1e3);
// NOTE: this is 1/1.6021766208e-19
Units::add("C", 6.241509126e18);
Units::add("fC", 6241.509126);
/label bug improvement