Added/updated __str__ methods for cleaning parameters

Removed doubled __eq__ methods for cleaning parameters

Signed-off-by: Daniel Madej <daniel.madej@intel.com>
This commit is contained in:
Daniel Madej 2021-08-31 07:39:04 +02:00
parent 70160debe2
commit 4f098b4695

View File

@ -143,6 +143,21 @@ class FlushParametersAlru:
and self.wake_up_time == other.wake_up_time and self.wake_up_time == other.wake_up_time
) )
def __str__(self):
ret = ["activity threshold: "
+ (f"{self.activity_threshold}" if self.activity_threshold is not None
else "default"),
"flush max buffers: "
+ (f"{self.flush_max_buffers}" if self.flush_max_buffers is not None
else "default"),
"staleness time: "
+ (f"{self.staleness_time}" if self.staleness_time is not None
else "default"),
"wake up time: "
+ (f"{self.wake_up_time}" if self.wake_up_time is not None
else "default")]
return " | ".join(ret)
@staticmethod @staticmethod
def alru_params_range(): def alru_params_range():
alru_params = FlushParametersAlru() alru_params = FlushParametersAlru()
@ -152,12 +167,6 @@ class FlushParametersAlru:
alru_params.wake_up_time = (0, 3600) alru_params.wake_up_time = (0, 3600)
return alru_params return alru_params
def __eq__(self, other):
return self.activity_threshold == other.activity_threshold and \
self.flush_max_buffers == other.flush_max_buffers and \
self.staleness_time == other.staleness_time and \
self.wake_up_time == other.wake_up_time
@staticmethod @staticmethod
def default_alru_params(): def default_alru_params():
alru_params = FlushParametersAlru() alru_params = FlushParametersAlru()
@ -180,12 +189,13 @@ class FlushParametersAcp:
) )
def __str__(self): def __str__(self):
ret = "" ret = ["flush max buffers: "
if self.flush_max_buffers is not None: + (f"{self.flush_max_buffers}" if self.flush_max_buffers is not None
ret += f"acp flush max buffers value: {self.flush_max_buffers} " else "default"),
if self.wake_up_time is not None: "wake up time: "
ret += f"acp wake up time value: {self.wake_up_time.total_milliseconds()}" + (f"{self.wake_up_time}" if self.wake_up_time is not None
return ret else "default")]
return " | ".join(ret)
@staticmethod @staticmethod
def acp_params_range(): def acp_params_range():
@ -194,10 +204,6 @@ class FlushParametersAcp:
acp_params.wake_up_time = (0, 10000) acp_params.wake_up_time = (0, 10000)
return acp_params return acp_params
def __eq__(self, other):
return self.flush_max_buffers == other.flush_max_buffers and \
self.wake_up_time == other.wake_up_time
@staticmethod @staticmethod
def default_acp_params(): def default_acp_params():
acp_params = FlushParametersAcp() acp_params = FlushParametersAcp()