dollabs / planviz

Planning Network Visualization
Apache License 2.0
14 stars 4 forks source link

Planviz support for name harmonization #55

Open dcerys opened 7 years ago

dcerys commented 7 years ago

For HTN nodes, and for TPN activities, use :display-name as the primary name displayed for that node/activity.

For legacy nodes/activities, we may want to use the previous attribute (e.g., :name) as a fallback in cases where :display-name isn't provided

Note that :display-name doesn't include any arguments. So, Planviz should provide a new user setting, with 3 options:

  1. Don't display any argument info (i.e., just the :display-name).
  2. Display the :args, after the :display-name, in a comma-delimited string, starting and ending with a parenthesis. E.g., Activity Name (Normal, 20, Left)
  3. Display the :argsmap, after the :display-name, in a comma-delimited string, starting and ending with a parenthesis. E.g., Activity Name (Intensity: Normal, Height: 20, Handedness: Left)

Of these 3, the default should be Option 2.

Full implementation of this will depend on the completion of some Pamela work (esp. pamela issue 43) [How do you link to another repository?]

dcerys commented 7 years ago

This needs to track the progress in https://github.com/dollabs/pamela/issues/43