diff --git a/Simulation/G4Atlas/G4AtlasServices/python/G4AtlasServicesConfigNew.py b/Simulation/G4Atlas/G4AtlasServices/python/G4AtlasServicesConfigNew.py
index 586b243b7237855119eba6c0d9957009458b2976..2e6e6cc22970084ffe14bfc820a561e3380faf18 100644
--- a/Simulation/G4Atlas/G4AtlasServices/python/G4AtlasServicesConfigNew.py
+++ b/Simulation/G4Atlas/G4AtlasServices/python/G4AtlasServicesConfigNew.py
@@ -1,24 +1,14 @@
 # Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration
 from AthenaConfiguration.ComponentAccumulator import ComponentAccumulator
 
-from AthenaCommon import CfgGetter
-#from G4AtlasApps.SimFlags import simFlags
-#from AthenaCommon.BeamFlags import jobproperties
-
 from G4AtlasServices.G4AtlasServicesConf import DetectorGeometrySvc, G4AtlasSvc, G4GeometryNotifierSvc
 #the physics region tools
 from G4AtlasTools.G4PhysicsRegionConfigNew import SX1PhysicsRegionToolCfg, BedrockPhysicsRegionToolCfg, CavernShaftsConcretePhysicsRegionToolCfg, PixelPhysicsRegionToolCfg, SCTPhysicsRegionToolCfg, TRTPhysicsRegionToolCfg, TRT_ArPhysicsRegionToolCfg, BeampipeFwdCutPhysicsRegionToolCfg, FWDBeamLinePhysicsRegionToolCfg, EMBPhysicsRegionToolCfg, EMECPhysicsRegionToolCfg, HECPhysicsRegionToolCfg, FCALPhysicsRegionToolCfg, DriftWallPhysicsRegionToolCfg, DriftWall1PhysicsRegionToolCfg, DriftWall2PhysicsRegionToolCfg, MuonSystemFastPhysicsRegionToolCfg
 
-
+#the geometry tools
 from G4AtlasTools.G4GeometryToolConfig import MaterialDescriptionToolCfg, G4AtlasDetectorConstructionToolCfg, ATLASEnvelopeCfg
+#the field config tools
 from G4AtlasTools.G4FieldConfigNew import ATLASFieldManagerToolCfg, TightMuonsATLASFieldManagerToolCfg, BeamPipeFieldManagerToolCfg, InDetFieldManagerToolCfg, MuonsOnlyInCaloFieldManagerToolCfg, MuonFieldManagerToolCfg, Q1FwdFieldManagerToolCfg, Q2FwdFieldManagerToolCfg, Q3FwdFieldManagerToolCfg, D1FwdFieldManagerToolCfg, D2FwdFieldManagerToolCfg, Q4FwdFieldManagerToolCfg, Q5FwdFieldManagerToolCfg, Q6FwdFieldManagerToolCfg, Q7FwdFieldManagerToolCfg, Q1HKickFwdFieldManagerToolCfg, Q1VKickFwdFieldManagerToolCfg, Q2HKickFwdFieldManagerToolCfg, Q2VKickFwdFieldManagerToolCfg, Q3HKickFwdFieldManagerToolCfg, Q3VKickFwdFieldManagerToolCfg, Q4VKickAFwdFieldManagerToolCfg, Q4HKickFwdFieldManagerToolCfg, Q4VKickBFwdFieldManagerToolCfg, Q5HKickFwdFieldManagerToolCfg,  Q6VKickFwdFieldManagerToolCfg, FwdRegionFieldManagerToolCfg
-#import G4AtlasTools.G4FieldConfigNew
-"""ClassicFieldManagerToolCfg
-BasicDetectorFieldManagerToolCfg
-BasicFwdFieldManagerToolCfg"""
-
-#already imported - is this correct?
-from MagFieldServices.MagFieldServicesConfig import MagneticFieldSvcCfg
 
 def getATLAS_RegionCreatorList(ConfigFlags):
     regionCreatorList = []
@@ -129,7 +119,6 @@ def ATLAS_FieldMgrListCfg(ConfigFlags):
     #if not simFlags.TightMuonStepping.statusOn or\
     #   not simFlags.TightMuonStepping():
     if False:
-        #fieldMgrList += [ATLASFieldManagerToolCfg(ConfigFlags)]
         acc   = ATLASFieldManagerToolCfg(ConfigFlags)
         tool  = result.popToolsAndMerge(acc)
         fieldMgrList += [tool]
@@ -160,28 +149,6 @@ def ATLAS_FieldMgrListCfg(ConfigFlags):
     #sort these forward ones later
     if ConfigFlags.Detector.SimulateForward: #needed?
         if ConfigFlags.Detector.GeometryFwdRegion: #or forward?
-          """fieldMgrList += [Q1FwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q2FwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q3FwdFieldManagerToolCfg(ConfigFlags),
-                                                           D1FwdFieldManagerToolCfg(ConfigFlags),
-                                                           D2FwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q4FwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q5FwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q6FwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q7FwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q1HKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q1VKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q2HKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q2VKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q3HKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q3VKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q4VKickAFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q4HKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q4VKickBFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q5HKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           Q6VKickFwdFieldManagerToolCfg(ConfigFlags),
-                                                           FwdRegionFieldManagerToolCfg(ConfigFlags)]"""
-
           accQ1FwdRegionFieldManager = Q1FwdFieldManagerToolCfg(ConfigFlags)
           accQ2FwdRegionFieldManager = Q2FwdFieldManagerToolCfg(ConfigFlags)
           accQ3FwdRegionFieldManager = Q3FwdFieldManagerToolCfg(ConfigFlags)
@@ -247,9 +214,9 @@ def ATLAS_FieldMgrListCfg(ConfigFlags):
                                                        toolQ5HKickFwdFieldManager,
                                                        toolQ6VKickFwdFieldManager,
                                                        toolFwdRegionFieldManager]
-          #fieldMgrList+=[toolQ1FwdRegionFieldManager]
+          
     result.setPrivateTools(fieldMgrList)
-    return result#, fieldMgrList
+    return result
 
 #called?
 def getCTB_FieldMgrList(ConfigFlags):
diff --git a/Simulation/G4Atlas/G4AtlasTools/python/G4FieldConfigNew.py b/Simulation/G4Atlas/G4AtlasTools/python/G4FieldConfigNew.py
index 84c5e55b76206f3e5797dcd8156cdf767a13f4fb..8eb1bcc1b1eada2054d8f2065c5759126f6e87e2 100644
--- a/Simulation/G4Atlas/G4AtlasTools/python/G4FieldConfigNew.py
+++ b/Simulation/G4Atlas/G4AtlasTools/python/G4FieldConfigNew.py
@@ -19,7 +19,7 @@ def ATLASFieldManagerToolCfg(ConfigFlags, name='ATLASFieldManager', **kwargs):
 
 
     result.setPrivateTools(GlobalFieldManagerTool(name, **kwargs))
-    return result#, GlobalFieldManagerTool(name, **kwargs)  # todo return accumulator 
+    return result
 
 def TightMuonsATLASFieldManagerToolCfg(ConfigFlags, name='TightMuonsATLASFieldManager', **kwargs):
     result = ComponentAccumulator()
@@ -35,12 +35,14 @@ def TightMuonsATLASFieldManagerToolCfg(ConfigFlags, name='TightMuonsATLASFieldMa
         kwargs.setdefault("EquationOfMotion", simFlags.EquationOfMotion.get_Value() )
 
     result.setPrivateTools(GlobalFieldManagerTool(name, **kwargs))
-    return result #, GlobalFieldManagerTool(name, **kwargs) #return tuple for now (add public / private tool later )
+    return result
 
+#not used in G4AtlasServicesConfigNew?
 def ClassicFieldManagerToolCfg(ConfigFlags, name='ClassicFieldManager', **kwargs):
     kwargs.setdefault("IntegratorStepper", "ClassicalRK4")
     return ATLASFieldManagerToolCfg(ConfigFlags, name, **kwargs)
 
+#not used in G4AtlasServicesConfigNew?
 def BasicDetectorFieldManagerToolCfg(ConfigFlags, name='BasicDetectorFieldManager', **kwargs):
     result = ComponentAccumulator()
     from G4AtlasApps.SimFlags import simFlags
@@ -94,7 +96,7 @@ def MuonFieldManagerToolCfg(ConfigFlags, name='MuonFieldManager', **kwargs):
     kwargs.setdefault('MinimumEpsilonStep', 0.000001)
     return BasicDetectorFieldManagerToolCfg(ConfigFlags, name, **kwargs)
 
-
+#not used in G4AtlasServicesConfigNew?
 def BasicFwdFieldManagerToolCfg(ConfigFlags, name='FwdFieldManagerTool', **kwargs):
     result=ComponentAccumulator()