[kyle-hoffmeyer]

TraceLens trace splitter invocation methods have been updated: AMD-AGI/TraceLens#1052.

Updating invocations in Magpie

[haofrank]

Could we preserve backward compatibility with existing TraceLens installations here by preferring TraceLens_split_trace and falling back to TraceLens_split_inference_trace when the new command is unavailable, for both local and container execution?

The rename is correct for newer TraceLens versions. The potential issue is upgrading Magpie while reusing an older TraceLens installation or cached Docker image: ensure_tracelens_installed() accepts an existing installation without checking for the new splitter command, and runtime preparation can reuse existing TraceLens images without upgrading them. Those environments would then fail during inference postprocessing because they only provide the old command.

Since upstream PR AMD-AGI/TraceLens#1052 preserves the splitter arguments and output format, a legacy-command fallback should keep these existing environments working. If supporting older versions is not intended, an explicit capability check and actionable upgrade/rebuild guidance would help make the new requirement clear.

[kyle-hoffmeyer]

@haofrank, thanks for your review. Since the new trace splitter is much more robust and can handle a variety of workloads, it'd be best if users migrate to the new trace splitter. Using the old trace splitter is much more likely to fail and produce incorrect outputs.
If the new trace splitter isn't found, the user will not get a message recommending the user to upgrade to the latest TraceLens release.

[haofrank]

OK, LGTM