Skip to content

Commit

Permalink
Fix text protocol buffer: place estimation_method at right level.
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
hzeller committed Oct 16, 2024
1 parent 49dfe42 commit 6a17244
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion place_and_route/private/benchmark.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -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),
),
),
)
Expand Down

0 comments on commit 6a17244

Please sign in to comment.