Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Scene Inventory tool: Refactor Scene Inventory tool (for AYON) #5758

Merged
merged 26 commits into from
Oct 13, 2023

Conversation

iLLiCiTiT
Copy link
Member

Changelog Description

Modified scene inventory tool for AYON. The main difference is in how project name is defined and replacement of assets combobox with folders dialog.

Additional info

Entity queries are still using functions from openpype.client. Usage of legacy_io was competelly removed. In switch dialog are used folder ids as unique identifier instead of name.

The dialog is needed because combobox did not support tree view and we need to support duplicated names of folders.

Screenshot

Questions

  • Folders field probably should have clear button?

Testing notes:

Basic tests

  1. Open host of your choice
  2. Load multiple representations
  3. Open scene inventory
  4. There should be all the loaded items
  5. Try to run some right-click actions
  6. Especially try switch dialog
    • Try to change Folder / subset name / representation name

@ynbot
Copy link
Contributor

ynbot commented Oct 13, 2023

Task linked: OP-5028 Scene Manager using AYON calls

1 similar comment
@ynbot
Copy link
Contributor

ynbot commented Oct 13, 2023

Task linked: OP-5028 Scene Manager using AYON calls

@ynbot ynbot added type: feature Larger, user affecting changes and completely new things size/XXL Denotes a PR changes 2500+ lines, ignoring general files labels Oct 13, 2023
Copy link
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

During testing I found that Switch Asset doesnt work...

No matter if not change anything just Asset or trying to redefine the Subset or Representation user cant confirm the switch action.

Irrelevant (wrong addon OP ver.)

image

@iLLiCiTiT
Copy link
Member Author

During testing I found that Switch Asset doesnt work...

I'm afraid you didn't update the openpype addon.

Copy link
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah now all good and works nicely!

Did try standard set of actions and all good!

@iLLiCiTiT iLLiCiTiT merged commit 70d8c72 into develop Oct 13, 2023
1 check passed
@iLLiCiTiT iLLiCiTiT deleted the feature/OP-5028_Scene-Manager-using-AYON-calls branch October 13, 2023 11:38
@ynbot ynbot added this to the next-patch milestone Oct 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/XXL Denotes a PR changes 2500+ lines, ignoring general files type: feature Larger, user affecting changes and completely new things
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants