Skip to content

Commit

Permalink
Switch to PrettyTable
Browse files Browse the repository at this point in the history
  • Loading branch information
sfvaroglu committed Jan 13, 2025
1 parent d7d5449 commit 14a59d6
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 11 deletions.
17 changes: 6 additions & 11 deletions .github/container/nsys_jax/nsys_jax/analyses/communication.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
load_profiler_data,
)
from math import sqrt
from prettytable import PrettyTable
from statistics import mean
import pathlib
from uncertainties import ufloat # type: ignore
Expand Down Expand Up @@ -104,25 +105,19 @@ def format_bandwidth(data, collective):
collective_types.add(collective)
summary_data[collective] = df["DurHiddenMsToDurMs"].mean()

collective_width = max(len("Collective"), max(len(f"{collective}") for collective in collective_types))
ratio_width = len("Mean HiddenToTotalMs")

print()
print(f"{'Collective':<{collective_width}} | {'Mean HiddenToTotalMs':<{ratio_width}}")
print(f"{'-' * collective_width} | {'-' * ratio_width}")
table = PrettyTable()
table.field_names = ["Collective", "Mean HiddenToTotalMs"]

for collective in collective_types:
mean_value = summary_data[collective]
collective_str = str(collective[0])
print(f"{collective_str:<{collective_width}} | {mean_value:>{ratio_width}}")
table.add_row([collective[0], mean_value])
print(table)

overall_hidden_ms_to_total_ms = (
steady_state.communication["ProjDurHiddenMs"].sum() /
(steady_state.communication["ProjDurMs"] + steady_state.communication["ProjDurHiddenMs"]).sum()
)

print()
print(f"Overall HiddenMs to TotalMs: {overall_hidden_ms_to_total_ms:>{ratio_width}}")
print(f"Overall HiddenMs to TotalMs: {overall_hidden_ms_to_total_ms}")

if __name__ == "__main__":
main()
1 change: 1 addition & 0 deletions .github/container/nsys_jax/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ dependencies = [
"pyarrow",
"requests", # for install-protoc
"uncertainties", # communication analysis recipe
"prettytable",
]
requires-python = ">= 3.10"

Expand Down

0 comments on commit 14a59d6

Please sign in to comment.