Enhancement: Houdini: Allow using template keys in Houdini shelves manager

[MustafaJafar]

Changelog Description

Allow using Template keys in Houdini shelves manager.

Testing notes:

1. add shelves or tools wile using template keys
2. check in houdini for their existence

[ynbot]

Task linked: OP-6710 Houdini Shelves Manager: pathes are not resolved if template keys are used

[ynbot]

Task linked: OP-6710 Houdini Shelves Manager: pathes are not resolved if template keys are used

[ynbot]

Task linked: OP-6710 Houdini Shelves Manager: pathes are not resolved if template keys are used

[moonyuet]

I have tested in Houdini. But for some reason the shelf doesn't show up on the shelf menu.

[MustafaJafar]

okay, here's what happens when creating a new shelf:

1. use shelf set path if exists and ignore everything, you should find it when clicking the + sign

Note that shelves manager ignores the set name in this case (which is a bug here)

2. create a shelf with the given shelf name if not exists and shelf set name and add the given tools

Note that it will save these data in the default shelf, so you may need to clean it.

your configuration should be split into two like this:

[moonyuet]

@MustafaJafar it works now after your guidance, thank you!

[MustafaJafar]

Would you use template keys to test this PR ?
I can tell from your screen shot that I forgot to allow using template keys with icon paths!

[moonyuet]

Looks good without the shelf_set_source_path

But I included the optional path shelf_set_source_path, the shelf menu doesn't show up in the list of shelves.
I need to create the shelf manually and named the shelf so that the script would be loaded in the shelf.

[MustafaJafar]

But I included the optional path shelf_set_source_path, the shelf menu doesn't show up in the list of shelves.
I need to create the shelf manually and named the shelf so that the script would be loaded in the shelf.

what about updating the algorithm of shelves manager as follows ?

1. if shelf_set_source_path only exists, then add it as a shelf in Houdini
2. if tools_list are provided without shelf_set_source_path, then create shelf with the given name in the default shelf.
3. if tools_list and shelf_set_source_path are provided, then create a shelf with the given path and add these tools to it. (however, I believe TDs can just create a shelf regularly then rely on the first case specified above.)

[moonyuet]

But I included the optional path shelf_set_source_path, the shelf menu doesn't show up in the list of shelves.
I need to create the shelf manually and named the shelf so that the script would be loaded in the shelf.

what about updating the algorithm of shelves manager as follows ?

1. if shelf_set_source_path only exists, then add it as a shelf in Houdini
2. if tools_list are provided without shelf_set_source_path, then create shelf with the given name in the default shelf.
3. if tools_list and shelf_set_source_path are provided, then create a shelf with the given path and add these tools to it. (however, I believe TDs can just create a shelf regularly then rely on the first case specified above.)

This is similar to the way how the shelves in maya. so I agree that we should do what you suggested

[MustafaJafar]

This is similar to the way how the shelves in maya. so I agree that we should do what you suggested

I added an issue to track it, Thanks!
#5738

[MustafaJafar]

@BigRoy Could you have a look? I'll appreciate it.

[BigRoy]

Didn't test the code - but the changes make sense. Looks good to me.

[antirotor]

It works and code looks good, even though setting the shelves up isn't very intuitive process.