TensorRT icon indicating copy to clipboard operation
TensorRT copied to clipboard

TREX(`trt-engine-explorer`): fix matadata typo (breaks report_card) and compatibility between dtale and dash

Open nimdrak opened this issue 3 months ago • 1 comments

I’m opening this issue in case it helps someone. If it’s useful, I’m happy to put together a PR.

Summary

In detail

1) Typo: matadata

There are a few occurrences of matadata in trex/report_card.py. This leads to a runtime error when calling report_card_draw_plan_graph_extended:

Example error stack (abbreviated):

.../trex/graphing.py:832 in to_dot
g = DotGraph(*args, **kwargs)
TypeError: DotGraph.__init__() got an unexpected keyword argument 'display_matadata'

Proposed fix Rename all matadata to metadata across TREX code where used.

2) dtaledash compatibility

dtale imports dash.development.component_loader, which was removed in dash 3.0+. With dash ≥ 3.0, this raises:

AttributeError: module 'dash.development' has no attribute 'component_loader'

Proposed fix We can fix it in two ways (1) Update the dtale version (fix https://github.com/man-group/dtale/issues/904)

# requirements-notebook.txt
dtale==3.18.2 # okay only if higher than 3.17.0

(2) Or pin dash to the last 2.x release in the notebook environment:

# requirements-notebook.txt
dash==2.18.2

This keeps dtale operational without changes.

nimdrak avatar Sep 12 '25 09:09 nimdrak

Hi @rajeevsrao, @asfiyab-nvidia — adding you for visibility since you’ve contributed a lot to this module. When you have a moment, could you take a look? (no rush) 🙇

nimdrak avatar Sep 12 '25 09:09 nimdrak