Investigate removing estimate method in CriterionBase

[cwschilly]

As mentioned in PR #546

CriterionBase defines an estimate method, but it may not be used.

• TemperedCriterion doesn't implement it at all (which causes a pylint warning)
• StrictLocalizingCriterion implements it by simply forwarding it to the compute function:

def estimate(self, r_src: Rank, o_src: list, *args) -> float:
        """Estimate is compute because all information is local for this criterion."""
        return self.compute(r_src, o_src, *args)

Investigate if we can just remove the estimate method entirely.