Upgrade TMA metrics for GNR

[harp-intel]

No description provided.

[Copilot]

Pull Request Overview

This PR upgrades TMA metrics for the Granite Rapids (GNR) platform by adding retire-latency support and refreshing scripts and definitions for event/metric translation and formatting.

• Add and load *_retire_latency.json data and replace [event:retire_latency] constants in expressions
• Enhance perfmonmetrics2perfspect.py and perfmonevents2perfspect.py with prefix stripping, parentheses checks, and improved CLI handling
• Update GNR metric JSON (gnr.json), retire-latency JSON, and event list TXT (gnr.txt) with new and revised metrics/events
• Inject inline-comment stripping in event_defs.go and wire TXN as an uncollectable event

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
scripts/perfmonmetrics2perfspect.py	Strip metric_ prefix, validate parentheses, and skip ill-formed metrics
scripts/perfmonevents2perfspect.py	Refactor format_event, inline usage message update, and optional file logic
scripts/check_events.py	Add "TXN" to core-event filter
cmd/metrics/metric_defs.go	Load retire latencies, replace [n] constants with regex, adjust transforms
cmd/metrics/event_defs.go	Strip end-of-line comments before parsing event definitions

Comments suppressed due to low confidence (2)

cmd/metrics/metric_defs.go:108

• [nitpick] The variable name reConstantInt is ambiguous; consider renaming it to something more descriptive like reBracketedNumber to clarify its purpose.

	reConstantInt := regexp.MustCompile(`\[(\d+)\]`)

scripts/perfmonmetrics2perfspect.py:76

• Indentation for the else block is inconsistent with the if branch above, causing a Python syntax error. Align this line to match the 8-space indent used in the if branch.

            metric["name"] = m["LegacyName"]