From 8657ee9bcec8d0da3612bfd5fd2e04c91ccaea90 Mon Sep 17 00:00:00 2001 From: Henner Zeller Date: Wed, 16 Oct 2024 13:06:20 -0700 Subject: [PATCH] Fix text protocol buffer: place `estimation_method` at right level. (#361) Make textproto parseable. According to [the schema](https://github.com/hdl/bazel_rules_hdl/blob/main/synthesis/power_performance_area.proto#L200), the estimation_method is part of the 'Power' message, not a sub-element of the PowerBreakdown message. Also: estimation_method is a string, so it needs to be quoted. --- place_and_route/private/benchmark.bzl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/place_and_route/private/benchmark.bzl b/place_and_route/private/benchmark.bzl index 4d247be0..7f62dd29 100644 --- a/place_and_route/private/benchmark.bzl +++ b/place_and_route/private/benchmark.bzl @@ -119,8 +119,8 @@ def benchmark(ctx, open_road_info): switching_watts = "$swi_pow", leakage_watts = "$leak_pow", total_watts = "$tot_pow", - estimation_method = "{} probabilistic switching fraction".format(ctx.attr.power_switching_activity), ), + estimation_method = "\"{} probabilistic switching fraction\"".format(ctx.attr.power_switching_activity), ), ), )