Removed the flag from AnalParams constructor.

[javierzaragoza]

This pull request addresses #2. Now the constructor reads the xml and decides if is beammap or science map what is requested. Since now it is possible to run the constructor for a beammap xml file from macanap and for a science xml from beammap, an error appears in macanap or beammap if the xml is not correct.

[Jerry-Ma]

The code looks right, I think it is good for now, thank you for the great contribution!

This part still have a lot room for improvement, though. Ideally, we would like to have a much better defined schema for the xml configuration files and verify the input (as well as input types) based on the schema. This will make the code much cleaner, and easily extendable.

If you are interested in improving coding techniques, implementing a schema based configuration verification/read/write would be a fine task. For our future Citlali code, we definitely want something like that.

I've used schema for a number of python projects before but not in C++. But I just did a quick googling and find this: https://www.codesynthesis.com/products/xsd . It looks like a good one, but there may be better options than this, but the idea should be similar.

If you have the time, you could try to do some more googling about existing solutions/libs and get a taste of them.

[Jerry-Ma]

I rebased your code to the tip of the master and resolved all the conflicts. I also updated the unit tests, and the PR is good to be merged.