Jopana
commented
7 years ago Hey David (escric en anglés per futura compatibilitat)
I will accept your pull request as it is, and then work on the code myself, to provide a new, bug-free version.
From now own, pull requests should be much smaller, with very specific changes, corrections or extensions. On top of that, I was considering in providing with a detailed documentation on the software through specific documentation platforms as "doxygen" or "sphinx".
As I told you at the beginning, we should do things in the most general, sharing and "good praxis" possible way, and try to learn from it.
I will post you back with the comments once I have a new working version. Quim
Aquesta és la versió que crec que funciona correctament, però no descarto que pugui haver-hi algun error que no hagi vist.
El codi està separat en tres classes.
Dark Matter class calcula el JFactor i la LOS.
Instrument class calcula la Epsilon i la Efficiency
Optimization class calcula les coses "d'interés". Estan inclosos els 3 casos: la Q que només inclou el background de la theta, la Q que inclou que inclou a més a més una "signal zone" i una "background zone" i la Q que inclou tot lo anterior però que està multiplicada per l'eficiència de la càmara.
He intentat comentar-ho el màxim possible perquè s'entengui bé, espero que sigui suficient.
Ja em diràs algo!