fix: arches should have no forward ecal or hcal inserts (#239)

### Briefly, what does this PR introduce?
This removes the forward ecal and hcal inserts from the `arches`
configuration.

It also (necessarily) introduces different handling for the hcal
systems, which until now were all included through `hcal.xml`. That
didn't allow for different hcal configurations. Now all hcal components
are included directly from the `epic.xml.jinja2` template.

### What kind of change does this PR introduce?
- [X] Bug fix (issue: arches should not have inserts)
- [ ] New feature (issue #__)
- [ ] Documentation update
- [ ] Other: __

### Please check if this PR fulfills the following:
- [ ] Tests for the changes have been added
- [ ] Documentation has been added / updated
- [X] Changes have been communicated to collaborators
- @johnlajoie This may affect hcal barrel inclusion since it modifies
the jinja template which you were editing too.
  - @rymilton @sly2j As discussed on Mattermost.

### Does this PR introduce breaking changes? What changes might users
need to make to their code?
No.

### Does this PR change default behavior?
Yes, `arches` will have no more inserts. `brycecanyon` has inserts.

configurations/arches.yml

@@ -18,11 +18,13 @@ features:
   ecal:
     forward_scifi:
     forward_homogeneous:
-    forward_insert:
     barrel_sciglass:
     backward_PbWO4:
   solenoid:
   hcal:
+    forward:
+    barrel:
+    backward:
   farforward:
     arches
   farbackward:

configurations/brycecanyon.yml

@@ -22,6 +22,10 @@ features:
     backward_PbWO4:
   solenoid:
   hcal:
+    forward:
+    forward_insert:
+    barrel:
+    backward:
   farforward:
     brycecanyon
   farbackward:

configurations/calorimeters.yml

@@ -8,3 +8,7 @@ features:
     barrel_sciglass
     backward_PbWO4
   hcal:
+    forward:
+    forward_insert:
+    barrel:
+    backward:

configurations/central.yml

@@ -18,3 +18,7 @@ features:
     backward_PbWO4
   solenoid:
   hcal:
+    forward:
+    forward_insert:
+    barrel:
+    backward:

configurations/full.yml

@@ -21,5 +21,9 @@ features:
     backward_PbWO4:
   solenoid:
   hcal:
+    forward:
+    forward_insert:
+    barrel:
+    backward:
   farforward:
   farbackward:

configurations/imaging.yml

@@ -21,5 +21,9 @@ features:
     backward_PbWO4:
   solenoid:
   hcal:
+    forward:
+    forward_insert:
+    barrel:
+    backward:
   farforward:
   farbackward:

configurations/sciglass.yml

@@ -21,5 +21,9 @@ features:
     backward_PbWO4:
   solenoid:
   hcal:
+    forward:
+    forward_insert:
+    barrel:
+    backward:
   farforward:
   farbackward:

templates/epic.xml.jinja2

@@ -171,7 +171,21 @@
   <documentation level="10">
     ## Central hadronic calorimetry
   </documentation>
-  <include ref="${DETECTOR_PATH}/compact/hcal.xml"/>
+  {% if features is not defined or features['hcal'] is none or 'forward' in features['hcal'] %}
+  <include ref="${DETECTOR_PATH}/compact/hcal/hcal_forward.xml"/>
+  {% endif -%}
+
+  {% if features is not defined or features['hcal'] is none or 'forward_insert' in features['hcal'] %}
+  <include ref="${DETECTOR_PATH}/compact/hcal/hcal_forward_insert.xml"/>
+  {% endif -%}
+
+  {% if features is not defined or features['hcal'] is none or 'barrel' in features['hcal'] %}
+  <include ref="${DETECTOR_PATH}/compact/hcal/hcal_barrel.xml"/>
+  {% endif -%}
+
+  {% if features is not defined or features['hcal'] is none or 'backward' in features['hcal'] %}
+  <include ref="${DETECTOR_PATH}/compact/hcal/hcal_backward.xml"/>
+  {% endif -%}
 {% endif -%}
 
 {% if features is not defined or 'beampipe' in features %}