Skip to content

SG-43444 [Phases 1-2] Migrate FlowAM integration#89

Open
carlos-villavicencio-adsk wants to merge 8 commits into
masterfrom
ticket/SG-43444_medm_migration
Open

SG-43444 [Phases 1-2] Migrate FlowAM integration#89
carlos-villavicencio-adsk wants to merge 8 commits into
masterfrom
ticket/SG-43444_medm_migration

Conversation

@carlos-villavicencio-adsk

@carlos-villavicencio-adsk carlos-villavicencio-adsk commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Move scene breakdown workflows previously being executed through flow framework directly into scene breakdown app.

  • Integration routing
    • manager hook invocations
    • file item model hook invocations and flowam conditionals
    • file history model
    • dialog clean up and minor improvements
  • New Hooks
    • get published files hook
  • Config hooks
    • flowam scene operations: Migrate the get_dependencies and update_dependency workflows into the app.

Config Requirements for FlowAM config basic

maya.apps.tk-multi-breakdown2:
  location: "@common.apps.tk-multi-breakdown2.location"
  # Disable panel model for easy reloading on development
  panel_mode: False
  • env/includes/houdini/apps.yml (basic)
  • env/includes/settings/tk-multi-breakdown2.yml (default2/advanced)
houdini.apps.tk-multi-breakdown2:
  location: "@common.apps.tk-multi-breakdown2.location"
  # Disable panel model for easy reloading on development
  panel_mode: False
  • env/includes/nuke/apps.yml (basic)
  • env/includes/settings/tk-multi-breakdown2.yml (default2/advanced)
nuke.apps.tk-multi-breakdown2:
  location: "@common.apps.tk-multi-breakdown2.location"
  # Disable panel model for easy reloading on development
  panel_mode: False

TODO

@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 17.29323% with 110 lines in your changes missing coverage. Please review.
✅ Project coverage is 28.73%. Comparing base (19cfce3) to head (485af16).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
python/tk_multi_breakdown2/file_history_model.py 9.61% 47 Missing ⚠️
python/tk_multi_breakdown2/file_item_model.py 3.33% 29 Missing ⚠️
python/tk_multi_breakdown2/api/manager.py 30.76% 27 Missing ⚠️
python/tk_multi_breakdown2/actions.py 0.00% 3 Missing ⚠️
python/tk_multi_breakdown2/dialog.py 0.00% 3 Missing ⚠️
app.py 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   29.34%   28.73%   -0.62%     
==========================================
  Files          19       19              
  Lines        2174     2290     +116     
==========================================
+ Hits          638      658      +20     
- Misses       1536     1632      +96     
Flag Coverage Δ
Linux 28.73% <17.29%> (-0.62%) ⬇️
Python-3.10 28.73% <17.29%> (-0.62%) ⬇️
Python-3.11 28.73% <17.29%> (-0.62%) ⬇️
Python-3.13 28.73% <17.29%> (-0.62%) ⬇️
Python-3.9 28.73% <17.29%> (-0.62%) ⬇️
Windows 28.73% <17.29%> (-0.62%) ⬇️
macOS 28.73% <17.29%> (-0.62%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@carlos-villavicencio-adsk carlos-villavicencio-adsk added the do not merge Do not merge Pull Request label Jun 10, 2026

@chenm1adsk chenm1adsk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Left some minor comments. The rest looks good to me!

Comment thread hooks/tk-maya_scene_operations.py
Comment thread hooks/flowam_scene_operations.py Outdated
Comment thread hooks/flowam_scene_operations.py Outdated
Comment thread hooks/flowam_scene_operations.py Outdated
Comment thread hooks/flowam_scene_operations.py Outdated

@chenm1adsk chenm1adsk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@carlos-villavicencio-adsk carlos-villavicencio-adsk changed the title SG-43444 [Part1] Migrate FlowAM integration SG-43444 [Phase 1] Migrate FlowAM integration Jun 19, 2026
* Removed `enable_flowam` Setting

* Remove framework references

* Add custom core for CI

* App testing and fixes

* Fix for CI

* Rename variables

* Add missing info on UI

* Improve UI values (created by, descriptions, etc)

* Add error display message

* Remove hook

* Refactor flowam hooks

* Fix CI

* Update comment

* Update .coveragerc file

* Revert .coveragerc changes

* Add codecov.yml file

* Fix entry

* Add conditionals to check host existence

* Fix attribute

* Fix bundle variable

* Improve handling for CI
@carlos-villavicencio-adsk carlos-villavicencio-adsk changed the title SG-43444 [Phase 1] Migrate FlowAM integration SG-43444 [Phases 1-2] Migrate FlowAM integration Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Do not merge Pull Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants