From 6b053c8185778ce4ac8eae0ca49463cf1a1a52f3 Mon Sep 17 00:00:00 2001 From: Hunter Nichols Date: Mon, 11 Jan 2021 12:53:14 -0800 Subject: [PATCH] Adjusted margin for the period in elmore model --- compiler/characterizer/elmore.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/compiler/characterizer/elmore.py b/compiler/characterizer/elmore.py index 7dbb4abd..b9f99f02 100644 --- a/compiler/characterizer/elmore.py +++ b/compiler/characterizer/elmore.py @@ -81,8 +81,9 @@ class elmore(simulation): else: debug.error("Measurement name not recognized: {}".format(mname), 1) - # Estimate the period as double the delay with margin - period_margin = 0.1 + # Margin for error in period. Calculated by averaging required margin for a small and large + # memory. FIXME: margin is quite large, should be looked into. + period_margin = 1.85 sram_data = {"min_period": (max_delay / 1e3) * 2 * period_margin, "leakage_power": power.leakage}