Skip to content
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

Loader - allow asset multiselection and bulk load #453

Open
mkolar opened this issue Sep 19, 2019 · 9 comments · May be fixed by #550
Open

Loader - allow asset multiselection and bulk load #453

mkolar opened this issue Sep 19, 2019 · 9 comments · May be fixed by #550

Comments

@mkolar
Copy link
Member

mkolar commented Sep 19, 2019

Problem
This is a repeating request from multiple clients to speed up set dressing. Usually when set dressing artists need to import a lot of assets at the same time especially when making cities forests and similar location which need a lot of individual assets with smaller variation in them. Currently this takes ages, becuase to import 10 trees (unless they are subsets) I need to click on asset, right click subset, load...repeat.

Proposed Solution
The solution would be to allow artist to select any number of assets on the left, which would show the union of all their available subsets on the right. The artist could import say 10 buildings at the same time. The problem of course is knowing what subsets belong to what asset and so on. We've tried solviing this with some grouping and color coding to make it clear what is going to be affected. I'm attaching a preview without much more explanation in hope that the UIX is clear enough on it's own, If it isn't we might want to think a bit more.

We've deployed this to testing now and the first feedback from artists is very positive. Have a look

multiselect

Additional context
Combined with #346 this might become mighty set dressing tool.

To make PR we'll first have to get in line with current state of avalon-core, which is almost done, so we have some time to discuss possible changes.

@mkolar
Copy link
Member Author

mkolar commented Sep 19, 2019

Just a closer look

closeup_multiselect

@BigRoy
Copy link
Collaborator

BigRoy commented Sep 19, 2019

Interesting. Visually it looks neat (I like that bar on the left hand-side). Nice job.

To understand correctly it's grouping the right hand side (Subsets) into 'randomly colored' groups per subset name so that still all the individual publishes of each asset can be seen. You are also unhiding an Asset column so there it's also clear from what asset each subset comes from.

I wonder how much speed-up this delivers as you work through it as an Artist. As I can imagine you'll still need to scavenge through the right-hand side. But I can imagine someone set dressing with solely models just selecting all assets on the left-hand side and and then limiting families to only model and solely browsing the right-hand side.

I'd love to try this out and maybe let some artists play to see if they have comments or remarks.


Also, I just wanted to link this video as a reference from Pixar on how they load setdress models. It could inspire more things.

@mkolar
Copy link
Member Author

mkolar commented Sep 19, 2019

To understand correctly it's grouping the right hand side (Subsets) into 'randomly colored' groups per subset name so that still all the individual publishes of each asset can be seen. You are also unhiding an Asset column so there it's also clear from what asset each subset comes from.

that's precisely what's going on.

I wonder how much speed-up this delivers as you work through it as an Artist. As I can imagine you'll still need to scavenge through the right-hand side.

To us a great speed up actually. We were just making a town and some forests. In both occasions we need to bring a lot of assets it and with this and consistent naming, it's a single import action. Selecting all assets that start with tree , all that start with bush (treeBirch', treeOak` etc) then right click on the modelMain group on the right side -> load abc -> done.

For trees for instance we'd have multiple variants as subsets in each asset, so Oak would have modelA modelB modelC and the same for other trees. If I'm bringing in 10 plant types and each has A,B,C variants that I can do it in one load instead of 30.

When we prepare the code to be compatible with core, we'd love for you to try it out and give more feedback of course.

@mkolar
Copy link
Member Author

mkolar commented Sep 19, 2019

Also, I just wanted to link this video as a reference from Pixar on how they load setdress models. It could inspire more things.

yeah that looks like USDs with proxy looks already applied. With avalon we'd normally first load models and and apply lookProxy to all. For rendering we'd change these to either standins or replace the looks with lookMain. Interesting thing is that they are technically importing the asset rather than subset in our terms, and it brings in the correct usd layer I assume. It's a bit different obviously because USD can include variants inside, whereas we only work with fairly rigid subsets.

@BigRoy
Copy link
Collaborator

BigRoy commented Sep 19, 2019

Interesting thing is that they are technically importing the asset rather than subset in our terms, and it brings in the correct usd layer I assume.

This particular tool could just be forcing to load a "model subset" of course, hence it being simplified. However, I'll try and refrain from going into that particular video too much and try to keep this issue discussion related to your approach.

@tokejepsen
Copy link
Collaborator

This feature could be interesting for making on-demand edits of different subsets.

@BigRoy
Copy link
Collaborator

BigRoy commented Sep 20, 2019

This feature could be interesting for making on-demand edits of different subsets.

What kind of on-demand edits would you want to be making to different subsets? Not sure I understand your use case. :)

@tokejepsen
Copy link
Collaborator

The workflow is to find the types of versions we want (animation, plate, lighting, compositing) in Ftrack, and RV (but any sequence player could do) to sequence them together for reviewing. With this feature we could do the same but in Avalon.

@mkolar
Copy link
Member Author

mkolar commented Nov 21, 2019

Just a note that this feature has been deployed to a few of our clients for a while now and the artist feedback is overwhelmingly positive. It's literally used on a daily basis, so we're very happy.

We'll prepare PR soon.

@iLLiCiTiT iLLiCiTiT linked a pull request May 26, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants