AcademySoftwareFoundation / AcademySoftwareFoundation/MaterialX

UINAME in nodedefs does not work when namespace is used

Open
#1,696 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
2.3k
Forks
451
Avg merge
6d 6h
Merged PRs (30d)
5

Description

Tried in MaterialX 1.38.8.

Steps to reproduce:
1 - Unzip and put the attached nodedefs file in your libraries.
2 - Start materiaX Graph Editor
3 - Press Tab to bring up list of nodes
4 - Select the "namespace_debug" group
5 - The node "nodedef_without_namespace" correctly uses the uiname string
6 - The node "test:ND_nodedef_with_namespace" does NOT use the uiname string.

The documentation for uiname in nodedefs specifically says:
"This is most useful when the defines a namespace, so the user doesn't need to see a full namespaced path for the node."
Which is the exact case it fails in this test.

[namespace_debug.zip](https://github.com/AcademySoftwareFoundation/MaterialX/files/14171020/namespace_debug.zip)

Please check also the video below. The video highlights 2 issues. This ticket is about the first part of the video.

https://github.com/user-attachments/assets/d578721c-c2da-4b2e-9e2b-b1f253f413fe

Contributor guide

Open the contributing guide

Research direction

Start by loading namespace_debug.zip in the MaterialX Graph Editor and reproducing the Tab-list behavior for the namespaced nodedef. Trace how UINAME is handled for namespaced versus non-namespaced nodedefs; done means the namespaced node displays its uiname instead of the full namespaced path.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-graphics
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.