TREX(`trt-engine-explorer`): fix matadata typo (breaks report_card) and compatibility between dtale and dash
I’m opening this issue in case it helps someone. If it’s useful, I’m happy to put together a PR.
Summary
- Fix the typo
matadata→metadatain report_card.py. - Update the
dtaleversion higher than 3.17.0 in requirements-notebook.txt to restore compatibility withdash.
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) dtale ↔ dash 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.
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) 🙇