-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Houdini: Load Alembic - disable automatic unpacking #253
Comments
Just adding @fabiaserra 's comment from here:
I do want to remark @fabiaserra that for more generalized "Loader UX" discussions/issue it's best to keep that separate from this particular issue. By the way, artists at our studio just confirmed: "It's fine to remove it." So we can also just kill this with 🔥 fire. I'd be happy to make a PR to clean that up if everyone's on board. @MustafaJafar @antirotor thoughts? |
I have a suggestion that lived for some time in shadows! (find it here on discord) Let's expose that in settings where pipeline admins can specify a default node tree. I think it's fairly simple to achieve as the structure in my example is as follows. name = "some-name-from-caller-code"
file_node = hou.node('/obj/box1/file1')
file_node.parent()
node_tree = {
"normal" : {"type": 0},
"null": {}
}
last_node = file_node
for node_type in node_tree:
node_name = "{}_{}".format(node_type, name)
node = file_node.parent().createNode(node_type, node_name=node_name)
node.setParms(node_tree[node_type])
node.setInput(0, last_node)
node.moveToGoodPosition()
last_node = node |
As far as I know, the loader UX is the same across all Hosts. which I assume makes it hard to add customizations per Host ? |
what I'm suggesting is not specific to Houdini, the same issue is present on all hosts |
I personally think that's too much exposed control without little benefit and quite unintuitive for the Ayon/OP admin, I'd rather be a bit more opinionated on the different possible efficient workflows through exposed options than leaving it open like this. I think I'd rather have a generic SOP HDA (like the LOP Ayon Load one that I've seen from @BigRoy) that can load any geo representation and has exposed options to control the internal graph of how to load it than having a tree of vanilla nodes. |
Just wanted to add that I agree. I wouldn't want to make up a new 'structure' based on JSON or whatever - it's just too limiting and too hard to set up as well. If someone wants a custom loader - they surely can, and it's really not that hard to implement. However, what AYON offers out of the box should be simple and not too specific to a certain way or working. So for now - I think removing the I think it's worth investigating whether there are good ways to make it customizable - but I think that's a separate goal from making sure what's already there is streamlined. (And likely a much more complex goal to keep things simple and streamlined for) |
Is there an existing issue for this?
Please describe the feature you have in mind and explain what the current shortcomings are?
Loading an Alembic into Houdini via the loader automatically adds an "Unpack" and "Normal" node which a long time ago was added for ease of access to the data (by artists at our studio).
However, @fabiaserra stated here that it might not be a good idea to keep that around and instead rely on automating less and instead leave it up to Houdini artists to manage the data. In essence, not creating those additional nodes on load.
How would you imagine the implementation of the feature?
Since it has been there for so long we might want to consider backwards compatibility. Just removing the logic for creation of those nodes should still work fine with existing loaded entries (they will just keep the nodes around) so it should be trivial to be 'forward' compatible.
The real question becomes, do we ever still want to keep the logic that it automatically creates it for you.
We could add it as "Options" on the Loader (little option box on the loader) or could make it a Project Settings to enable/disable the logic.
Are there any labels you wish to add?
Describe alternatives you've considered:
No response
Additional context:
No response
[cuID:AY-4796]
The text was updated successfully, but these errors were encountered: