ok, that makes sense; I think store is a way better place to worry about whether errors are encodable or not than benchmarker, however we might be able to make this much simpler... why don't we just make the Error field in Sample/Store etc. a string (since that's actually all we care about at those points)? Then we can totally avoid all this silly EncodableError stuff..