(cherry picked from commit 1ecc6e58)

(cherry picked from commit 2e853a98)

- When there is activity, this causes the master pipeline to never
finish. Not good.
- ip6 triggers cancel current pipeline too

Paying attention to following items:
* Overall parameterization -- will use for optimization
* Constant naming consistency
* Documentation via the comments
* Added new ci job which concatenates all the `<comment>` tags into `docs/detector.md`
* All comments should now be formatted to fit the appropriate section in the markdown file.

Needs to be changed when we fix the ACTS issues.

	modified:   .gitlab-ci.yml
	modified:   compact/solenoid.xml

Fixed ce_GEM overlap

- add job in CI and saving output to doc/constants.out

Not sure why, but before_script seems to always succeed even if a step fails.

	new file:   compact/forward_rich.xml
	modified:   reference_detector.xml
	new file:   src/ForwardRICH_geo.cpp