Subclasses of AttributeMapper only have to specify what's new about their requirements; if they inherit from something like BaseDiscretization and it has an initMap attribute, they will automatically inherit whichever keys they don't override.
The key initialization code runs before__init__ in the subclass, which means that it is not necessary to call super(SubClass, self).__init__(systemConfig, *args, **kwargs) just to add some code to __init__; though, fair to say, it may still be a good idea depending on the circumstances.
Suggest merging whenever we're comfortable with the changes.
bsmithyman
commented
8 years ago 
Overall, I think this is an improvement. It adds:
AttributeMapperonly have to specify what's new about their requirements; if they inherit from something likeBaseDiscretizationand it has aninitMapattribute, they will automatically inherit whichever keys they don't override.__init__in the subclass, which means that it is not necessary to callsuper(SubClass, self).__init__(systemConfig, *args, **kwargs)just to add some code to__init__; though, fair to say, it may still be a good idea depending on the circumstances.Suggest merging whenever we're comfortable with the changes.
bsmithyman included the following code: https://github.com/uwoseis/anemoi/pull/1/commits