Adding contrite TfT

[drvinceknight]

Closes #636

A strategy that plays like TfT if there is no noise. If there is noise and it defects because of noise (intending to cooperate), if the opponent cooperates but defects as a result of the noisy defection then it cooperates to try and bring back mutual cooperation.

Wasn't sure if there was a better way to handling the renaming from the TrackHistoryTransformer (without my change it adds a prefix to the strategy)?

Note that this is on top of #638 (as it needed that fix to work for the 'reduce to tft' test to work).

[meatballs]

Just a typo in the comment - 'and but'

[drvinceknight]

Fixing now :)

[marcharper]

Wasn't sure if there was a better way to handling the renaming from the TrackHistoryTransformer (without my change it adds a prefix to the strategy)?

You could just make a new instance:

transformer = StrategyTransformerFactory(
    history_track_wrapper, name_prefix="")()

The prefix is there to prevent overwriting the original class; in this case it doesn't matter but it may in other contexts. I'd prefer to leave the default with some prefix. Actually, the transformer is supposed to take a name_prefix parameter:

        def __call__(self, PlayerClass):
            """
            Parameters
            ----------
            PlayerClass: A subclass of axelrod.Player, e.g. Cooperator
                The Player Class to modify
            name_prefix: str
                A string to prepend to the Player and Class name

so that the following would work:

@TrackHistoryTransformer(name_prefix='')
class MyPlayer(object):
    name = "MyName"
print(MyPlayer().name)

but as you can see it's not implemented properly.

[drvinceknight]

Thanks: have fixed that using your suggestion.

The prefix is there to prevent overwriting the original class; in this case it doesn't matter but it may in other contexts. I'd prefer to leave the default with some prefix.

Me too, I like having the prefix. Would be great to be able to pass the prefix (or None) to the decorator as you said there. I'll open an issue for it (also, note that FinalTransformer does not have a prefix (used by Backstabber). That should be addressed too.

[drvinceknight]

Have opened #642