Refactored based on Dan and Zyed's comments

Pretty pedantic, but if you're allowing the user to set `summary_values` via `attr_accessor` anyway, may as well provide a default argument to the constructor here so that they can construct the object without data and set it later.