Skip to content

Metrics fix for short_dict #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Dimosts merged 2 commits into from
May 13, 2025
Merged

Metrics fix for short_dict #13

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f117cea
short dict
Dimosts May 13, 2025
b9c0d84
Update metrics.py
Dimosts May 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions pycona/metrics.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,14 +111,14 @@ def increase_queries_size(self, amount):
"""
self.total_size_queries += amount

def aggreagate_max_waiting_time(self, max2):
def aggreagate_max_waiting_time(self, t):
"""
Aggregate the maximum waiting time.

:param max2: The new waiting time to compare with the current maximum.
:param t: The new waiting time to compare with the current maximum.
"""
if self.max_waiting_time < max2:
self.max_waiting_time = max2
if self.max_waiting_time < t:
self.max_waiting_time = t

def increase_recommendation_queries_count(self, amount=1):
"""
Expand Down Expand Up @@ -252,7 +252,7 @@ def print_short_statistics(self):
results_df = pd.DataFrame([self.metrics_short_dict])
print(results_df.to_string(index=False))
except ImportError:
print(self.metrics_dict)
print(self.metrics_short_dict)

@property
def short_statistics(self):
Expand All @@ -264,7 +264,7 @@ def short_statistics(self):
results_df = pd.DataFrame([self.metrics_short_dict])
return results_df
except ImportError:
return self.metrics_dict
return self.metrics_short_dict

def write_to_file(self, filename):
"""
Expand Down